Rename pelican-dev org references to pelican - #208
Merged
notAreYouScared merged 1 commit intoAug 16, 2026
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Everything left after the module rename: the --repo-owner default that self-update uses to find releases, the CODEOWNERS team, the image in the compose example, and the links in the README, SECURITY and CHANGELOG.
lancepioch
force-pushed
the
chore/org-rename-pelican
branch
from
August 14, 2026 23:34
779980a to
96636e9
Compare
notAreYouScared
marked this pull request as ready for review
August 16, 2026 21:16
This was referenced Aug 16, 2026
lancepioch
added a commit
that referenced
this pull request
Aug 16, 2026
2 tasks
engels74
added a commit
to engels74/wings-vpn
that referenced
this pull request
Aug 18, 2026
## Upstream sync: `upstream/main@090841d` | Field | Value | | --- | --- | | Commits merged | 2 | | Files changed | 98 (+254 / -254) | | Conflicts | 1 (1 auto-resolved, 0 manual) | | Status | Ready for review | ### Summary This sync brings in two upstream commits that finish the wings organization rename, including the Go module path change to github.com/pelican/wings. It touches 98 files with 254 insertions and 254 deletions, and one conflicted file was resolved automatically. ### Conflict resolution | File | Type | Resolution | | --- | --- | --- | | `README.md` | both modified | Combined fork title and documentation with upstream's updated GitHub Issues URL. | <details> <summary>Per-file detail</summary> **`README.md`** — both modified - Upstream: Upstream updated the GitHub Issues link from github.com/pelican-dev/wings to github.com/pelican/wings. - Fork: The fork changed the README title to include a wings-vpn fork label and added an About this fork section documenting container network mode. - Resolution: Combined fork title and documentation with upstream's updated GitHub Issues URL. </details> <details> <summary>Merged commits (2)</summary> - `090841d` Finish the wings org rename (the part pelican#208 missed) (pelican#212) - `5bd4d7e` Rename the Go module to github.com/pelican/wings (pelican#209) </details> <details> <summary>Changed files (98)</summary> ```` .github/CODEOWNERS | 2 +- .github/workflows/push.yaml | 4 ++-- .github/workflows/release.yaml | 4 ++-- CHANGELOG.md | 4 ++-- Dockerfile | 2 +- Makefile | 4 ++-- README.md | 2 +- SECURITY.md | 4 ++-- cmd/configure.go | 2 +- cmd/diagnostics.go | 4 ++-- cmd/root.go | 22 +++++++++++----------- cmd/selfupdate.go | 4 ++-- config/config.go | 2 +- docker-compose.example.yml | 2 +- environment/allocations.go | 2 +- environment/docker.go | 2 +- environment/docker/api.go | 2 +- environment/docker/container.go | 6 +++--- environment/docker/environment.go | 8 ++++---- environment/docker/power.go | 4 ++-- environment/docker/stats.go | 2 +- environment/environment.go | 2 +- environment/settings.go | 2 +- events/events.go | 2 +- go.mod | 2 +- internal/cron/activity_cron.go | 8 ++++---- internal/cron/cron.go | 6 +++--- internal/cron/sftp_cron.go | 8 ++++---- internal/database/database.go | 6 +++--- internal/diagnostics/logs.go | 6 +++--- internal/progress/progress.go | 2 +- internal/progress/progress_test.go | 2 +- internal/ufs/fs_platform_test.go | 2 +- internal/ufs/fs_unix_test.go | 2 +- loggers/cli/cli.go | 8 ++++---- parser/parser.go | 4 ++-- remote/http.go | 4 ++-- remote/servers.go | 2 +- remote/types.go | 2 +- router/downloader/downloader.go | 4 ++-- router/middleware.go | 4 ++-- router/middleware/middleware.go | 8 ++++---- router/middleware/request_error.go | 4 ++-- router/router.go | 8 ++++---- router/router_download.go | 6 +++--- router/router_server.go | 16 ++++++++-------- router/router_server_backup.go | 8 ++++---- router/router_server_backup_test.go | 12 ++++++------ router/router_server_files.go | 16 ++++++++-------- router/router_server_files_search.go | 10 +++++----- router/router_server_transfer.go | 10 +++++----- router/router_server_ws.go | 6 +++--- router/router_system.go | 14 +++++++------- router/router_transfer.go | 12 ++++++------ router/tokens/parser.go | 2 +- router/websocket/listeners.go | 6 +++--- router/websocket/websocket.go | 16 ++++++++-------- server/activity.go | 4 ++-- server/backup.go | 6 +++--- server/backup/backup.go | 6 +++--- server/backup/backup_local.go | 6 +++--- server/backup/backup_s3.go | 6 +++--- server/backup/backup_test.go | 4 ++-- server/config_parser.go | 2 +- server/configuration.go | 2 +- server/connections.go | 2 +- server/console.go | 4 ++-- server/crash.go | 6 +++--- server/events.go | 4 ++-- server/filesystem/archive.go | 6 +++--- server/filesystem/compress.go | 4 ++-- server/filesystem/disk_space.go | 2 +- server/filesystem/errors.go | 2 +- server/filesystem/filesystem.go | 4 ++-- server/filesystem/filesystem_test.go | 4 ++-- server/filesystem/path_test.go | 2 +- server/filesystem/quota_file.go | 2 +- server/filesystem/quotas/exfs.go | 2 +- server/filesystem/quotas/functions.go | 2 +- server/filesystem/stat.go | 2 +- server/install.go | 8 ++++---- server/installer/installer.go | 4 ++-- server/listeners.go | 8 ++++---- server/manager.go | 12 ++++++------ server/mounts.go | 4 ++-- server/power.go | 6 +++--- server/power_test.go | 2 +- server/resources.go | 8 ++++---- server/server.go | 16 ++++++++-------- server/transfer/archive.go | 6 +++--- server/transfer/source.go | 2 +- server/transfer/transfer.go | 4 ++-- server/update.go | 4 ++-- sftp/event.go | 4 ++-- sftp/handler.go | 6 +++--- sftp/handler_test.go | 2 +- sftp/server.go | 6 +++--- wings.go | 2 +- 98 files changed, 254 insertions(+), 254 deletions(-) ```` </details> ### Review checklist - [ ] Conflict resolutions preserve fork-specific behaviour - [ ] `make build` and `make test` pass <!-- greptile_comment --> <details><summary><h3>Greptile Summary</h3></summary> This PR completes the upstream Pelican organization and Go module rename while preserving the fork-specific README content. - Renames the module and internal imports from `github.com/pelican-dev/wings` to `github.com/pelican/wings`. - Updates build workflows, linker paths, release links, image references, and self-update defaults to the new organization. - Keeps the fork issue tracker and container-network-mode documentation in `README.md`. </details> <details><summary><h3>Confidence Score: 5/5</h3></summary> The PR appears safe to merge, with no changed-code-triggered defects identified. The module declaration, source imports, build targets, linker flags, release configuration, and organization references are consistently renamed, while the fork-specific README resolution is preserved. </details> <details><summary><h3>Important Files Changed</h3></summary> | Filename | Overview | |----------|----------| | go.mod | Renames the Go module to `github.com/pelican/wings`, consistently matching the updated imports and build configuration. | | cmd/selfupdate.go | Updates the default upstream release owner to `pelican`; artifact naming remains aligned with the release workflow. | | .github/workflows/push.yaml | Updates build package and linker-symbol paths to the renamed Go module. | | .github/workflows/release.yaml | Updates release build and version linker paths while retaining the existing Linux artifact contract. | | Dockerfile | Updates the version linker target to the renamed module path. | | docker-compose.example.yml | Updates the existing upstream container image reference to the renamed Pelican organization. | | README.md | Preserves fork-specific documentation and adds the renamed upstream issue tracker. | </details> <sub>Reviews (1): Last reviewed commit: ["sync upstream/main"](098a3c1) | [Re-trigger Greptile](https://app.greptile.com/api/retrigger?id=53900332)</sub> <!-- /greptile_comment -->
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Do not merge this yet. This is the half that genuinely has to wait, unlike #209.
The module path rename moved out into #209, which is safe to land whenever, so what's left here is just the six lines that are live values pointing at a place that doesn't exist yet. Based on #209 for now so the diff only shows the remainder; it'll retarget itself to main once that one goes in.
The one worth actually looking at is
cmd/selfupdate.go, where--repo-ownerdefaults topelican-dev. That's the value self-update uses to go looking for releases, and it's baked into binaries that are already out in the wild, so a fresh checkout doesn't help anybody who already has wings installed.The rest is the CODEOWNERS team, the image in
docker-compose.example.yml, and links in the README, SECURITY and CHANGELOG.Two things that can't be done from a PR and still need doing separately: recreate the
@pelican/wingsteam so CODEOWNERS resolves again, and republish the wings image underghcr.io/pelican/wings.