Repository navigation
compute: Add help text for default value of --version when it is optional - #1928
Merged
Merged
Conversation
Member
|
The proper fix would be to make the |
kpfleming
approved these changes
Oct 5, 2026
harmony7
force-pushed
the
kats/version-flag-help
branch
from
October 10, 2026 06:22
cc7173e to
1256415
Compare
harmony7
enabled auto-merge (squash)
October 10, 2026 06:22
harmony7
disabled auto-merge
October 10, 2026 06:28
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
harmony7
force-pushed
the
kats/version-flag-help
branch
from
October 10, 2026 06:28
1256415 to
bc3cc74
Compare
harmony7
enabled auto-merge (squash)
October 10, 2026 06:29
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.
Change summary
This PR updates the help text for
--versionwhen it's optional, to describe what the default value is.Before this PR, the help text looks like this:
For commands like
compute deployandcompute publish,--versionis optional, but it'sunclear from the help text what happens when you don't pass it.
After this PR, the help text for version looks like this:
I felt it would be helpful to document this default behavior because I found myself reading source to find out what happens when
--versionis not passed in (for example how fastly/compute-actions behaves under default conditions)All Submissions:
New Feature Submissions:
Changes to Core Features:
(There are no new tests for this because there are no tests that check flag descriptions.)
User Impact
Should be no impact unless someone relies on the output of help text
Are there any considerations that need to be addressed for release?
None