Skip to content

Finish the wings org rename (the part #208 missed) - #211

Closed
lancepioch wants to merge 2 commits into
mainfrom
chore/wings-module-path
Closed

Finish the wings org rename (the part #208 missed)#211
lancepioch wants to merge 2 commits into
mainfrom
chore/wings-module-path

Conversation

@lancepioch

@lancepioch lancepioch commented Aug 16, 2026

Copy link
Copy Markdown
Member

#208 never actually reached main. It was based on chore/wings-module-path while that was still open, and because that branch didn't get deleted after #209 merged, GitHub left the base alone and #208 merged into the branch instead of into main. So the module rename landed and these six lines didn't.

This is the same content, retargeted at main. Nothing new in it.

The one that matters is cmd/selfupdate.go, where --repo-owner is still pelican-dev — that's what self-update uses to look for releases. The rest is the CODEOWNERS team, the image in docker-compose.example.yml, and links in the README, SECURITY and CHANGELOG.

Summary by CodeRabbit

  • Chores
    • Updated project references to the new Pelican Wings repository location.
    • Updated build, release, Docker, self-update, and container image configurations.
    • Refreshed documentation, support links, security resources, and changelog references.
    • Existing runtime behavior and user-facing functionality remain unchanged.

lancepioch and others added 2 commits August 13, 2026 23:05
Nothing outside this repo imports wings, so the imports are all
self-referential and this builds the same either way; the point is just
to have the module path match where the code lives.

The version stamping has to agree with the module path or it silently
stops working, so SRC_PATH, the release ldflags, the Dockerfile and the
Makefile move in the same commit.
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

Copy link
Copy Markdown
Member Author

Wrong base — this replays the module rename that's already on main. Redoing it as a clean branch off current main.

@lancepioch lancepioch closed this Aug 16, 2026
@coderabbitai

coderabbitai Bot commented Aug 16, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 473f314f-6513-4e2f-a84f-b98c42861039

📥 Commits

Reviewing files that changed from the base of the PR and between 5bd4d7e and 7eede1f.

📒 Files selected for processing (98)
  • .github/CODEOWNERS
  • .github/workflows/push.yaml
  • .github/workflows/release.yaml
  • CHANGELOG.md
  • Dockerfile
  • Makefile
  • README.md
  • SECURITY.md
  • cmd/configure.go
  • cmd/diagnostics.go
  • cmd/root.go
  • cmd/selfupdate.go
  • config/config.go
  • docker-compose.example.yml
  • environment/allocations.go
  • environment/docker.go
  • environment/docker/api.go
  • environment/docker/container.go
  • environment/docker/environment.go
  • environment/docker/power.go
  • environment/docker/stats.go
  • environment/environment.go
  • environment/settings.go
  • events/events.go
  • go.mod
  • internal/cron/activity_cron.go
  • internal/cron/cron.go
  • internal/cron/sftp_cron.go
  • internal/database/database.go
  • internal/diagnostics/logs.go
  • internal/progress/progress.go
  • internal/progress/progress_test.go
  • internal/ufs/fs_platform_test.go
  • internal/ufs/fs_unix_test.go
  • loggers/cli/cli.go
  • parser/parser.go
  • remote/http.go
  • remote/servers.go
  • remote/types.go
  • router/downloader/downloader.go
  • router/middleware.go
  • router/middleware/middleware.go
  • router/middleware/request_error.go
  • router/router.go
  • router/router_download.go
  • router/router_server.go
  • router/router_server_backup.go
  • router/router_server_backup_test.go
  • router/router_server_files.go
  • router/router_server_files_search.go
  • router/router_server_transfer.go
  • router/router_server_ws.go
  • router/router_system.go
  • router/router_transfer.go
  • router/tokens/parser.go
  • router/websocket/listeners.go
  • router/websocket/websocket.go
  • server/activity.go
  • server/backup.go
  • server/backup/backup.go
  • server/backup/backup_local.go
  • server/backup/backup_s3.go
  • server/backup/backup_test.go
  • server/config_parser.go
  • server/configuration.go
  • server/connections.go
  • server/console.go
  • server/crash.go
  • server/events.go
  • server/filesystem/archive.go
  • server/filesystem/compress.go
  • server/filesystem/disk_space.go
  • server/filesystem/errors.go
  • server/filesystem/filesystem.go
  • server/filesystem/filesystem_test.go
  • server/filesystem/path_test.go
  • server/filesystem/quota_file.go
  • server/filesystem/quotas/exfs.go
  • server/filesystem/quotas/functions.go
  • server/filesystem/stat.go
  • server/install.go
  • server/installer/installer.go
  • server/listeners.go
  • server/manager.go
  • server/mounts.go
  • server/power.go
  • server/power_test.go
  • server/resources.go
  • server/server.go
  • server/transfer/archive.go
  • server/transfer/source.go
  • server/transfer/transfer.go
  • server/update.go
  • sftp/event.go
  • sftp/handler.go
  • sftp/handler_test.go
  • sftp/server.go
  • wings.go

📝 Walkthrough

Walkthrough

The pull request renames the Go module and repository references from pelican-dev/wings to pelican/wings. It updates imports, build commands, workflows, container references, documentation links, ownership, and the self-update repository default.

Changes

Module path migration

Layer / File(s) Summary
Module contract and build references
.github/workflows/*, Dockerfile, Makefile, go.mod, cmd/selfupdate.go, .github/CODEOWNERS
Build commands, release commands, linker flags, ownership, and self-update settings use pelican/wings.
Command and foundation imports
cmd/*, config/*, environment/*, events/*, internal/*, loggers/*, parser/*, remote/*
Internal Go imports now use github.com/pelican/wings.
Router and server imports
router/*, server/*
Router, server, backup, filesystem, installer, and transfer imports now use github.com/pelican/wings.
Repository integrations and tests
docker-compose.example.yml, README.md, SECURITY.md, CHANGELOG.md, sftp/*, wings.go
Container images, repository links, changelog links, SFTP imports, and the entrypoint import use the new repository path.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: quintenqvd0

Poem

A rabbit hops through paths anew,
From dev to pelican, clean and true.
Builds and links now point the way,
Imports follow without delay.
“Sniff,” says Bun, “the names align!”

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/wings-module-path

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@lancepioch
lancepioch deleted the chore/wings-module-path branch August 16, 2026 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant