Skip to content

CDTOOL-1264: Introduce 'service version ...' commands and 'service-version ...' aliases.#1614

Merged
kpfleming merged 2 commits intofastly:mainfrom
kpfleming:CDTOOL-1264
Jan 15, 2026
Merged

CDTOOL-1264: Introduce 'service version ...' commands and 'service-version ...' aliases.#1614
kpfleming merged 2 commits intofastly:mainfrom
kpfleming:CDTOOL-1264

Conversation

@kpfleming
Copy link
Contributor

@kpfleming kpfleming commented Jan 12, 2026

Change summary

This PR adds the 'service version ...' commands as a replacement for the 'service-version ...' commands. The 'service-version ...' commands are still available as aliases, but will issue a deprecation warning when used (except when JSON output is requested).

This involved a number of changes:

  1. The existing 'pkg/commands/serviceversion' package content was moved to 'pkg/commands/service/version', and modified to work as expected in that location.
  2. An alias package was added in 'pkg/commands/alias/serviceversion'. This alias package uses struct embedding to avoid duplicating any of the code from the `pkg/commands/service/version' package. The alias commands are 'unlisted' and issue a deprecation warning when used.

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  • Does your submission pass tests?

User Impact

New deprecation warning for the service-version ... commands.

@kpfleming kpfleming marked this pull request as ready for review January 12, 2026 21:45
@kpfleming kpfleming requested a review from a team as a code owner January 12, 2026 21:45
Rename 'fastly service-version ...' to 'fastly service version...'.
Add hidden and deprecated 'fastly service-version' aliases for
'fastly service version' commands.
@kpfleming
Copy link
Contributor Author

Rebased on current 'main', no other changes.

Copy link
Contributor

@anthony-gomez-fastly anthony-gomez-fastly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kpfleming kpfleming enabled auto-merge (squash) January 15, 2026 16:08
@kpfleming kpfleming merged commit 9c1f02f into fastly:main Jan 15, 2026
9 checks passed
@kpfleming kpfleming deleted the CDTOOL-1264 branch January 15, 2026 16:12
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.

2 participants