Skip to content

refactor: move verb packages under internal/cmd - #465

Open
thde wants to merge 1 commit into
thde/352-07-testutilfrom
thde/352-08-internal-cmd
Open

thde wants to merge 1 commit into
thde/352-07-testutilfrom
thde/352-08-internal-cmd

Conversation

@thde

@thde thde commented Sep 17, 2026

Copy link
Copy Markdown
Member

Move the ten CLI verb packages (apply, auth, copy, create, delete, edit, exec, get, logs, update) from the module root to internal/cmd/. The package names and their code are unchanged; only the import paths in main.go move along.

TestVerbsDoNotImportEachOther now resolves the verb directories and the verb import prefix from a verbDir constant so it keeps guarding the no-verb-imports-verb rule at the new location.

Last step of #352.

@thde
thde added this pull request to stack #459 September 17, 2026 12:11
@thde
thde force-pushed the thde/352-08-internal-cmd branch from 7112667 to f6040fd Compare October 6, 2026 08:35
Move the ten CLI verb packages (apply, auth, copy, create, delete, edit,
exec, get, logs, update) from the module root to internal/cmd/<verb>.
The
package names and their code are unchanged; only the import paths in
main.go move along.

TestVerbsDoNotImportEachOther now resolves the verb directories and the
verb import prefix from a verbDir constant so it keeps guarding the
no-verb-imports-verb rule at the new location.

Last step of #352.

# Conflicts:
#	internal/cmd/get/database.go
#	internal/cmd/get/database_test.go
#	internal/cmd/get/service.go
#	internal/cmd/get/service_test.go
@thde
thde force-pushed the thde/352-08-internal-cmd branch from f6040fd to 1240cca Compare October 9, 2026 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant