-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixing #8043 - adding long form commands and aliasing abbreviations #8073
Fixing #8043 - adding long form commands and aliasing abbreviations #8073
Conversation
…bbreviations Signed-off-by: Nick Beenham <Nicholas_Beenham@cable.comcast.com>
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was going to ask if we should update examples like this one
radius/pkg/cli/cmd/env/show/show.go
Line 49 in 7f2ee9f
rad env show |
rad app
examples either, so it likely is fine as is at this point.
@superbeeny, looks like some of the docs references would need to be manually updated https://github.com/radius-project/docs/actions/runs/11942519605/job/33289744491?pr=1289. @Reshrahim, how do you recommend resolving this given the circular dependency between this pr radius-project/docs#1289 and the manual updates needed? |
yes just noticed the same thing. Easy way is to update the references in the same PR -radius-project/docs#1289 |
…bbreviations (radius-project#8073) # Description Updated cmd line functions to use `environment` in place of `env` and `initialize` in place of `init`. Both abbreviations are now aliased so this is a non breaking change. ## Type of change - This pull request fixes a bug in Radius and has an approved issue (issue link required). Fixes: radius-project#8043 Signed-off-by: Nick Beenham <Nicholas_Beenham@cable.comcast.com>
Description
Updated cmd line functions to use
environment
in place ofenv
andinitialize
in place ofinit
. Both abbreviations are now aliased so this is a non breaking change.Type of change
Fixes: #8043
Contributor checklist
Please verify that the PR meets the following requirements, where applicable: