This repository has been archived by the owner on Mar 9, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 23
[kn-admin] Refine subcommand wording #53
Comments
/assign |
@chaozbj |
@zhanggbj ok, no prblem |
I would like to put it on hold because kn client has the similar feature and waits for implementation: knative/client#655, we will follow kn client to introduce create/apply/remove in our plugin. |
The codes have been merged, will close it. |
@chaozbj: You can't close an active issue/PR unless you authored it or you are a collaborator. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Per discussion in
#40 (comment)
now in kn-admin, for different subcommand domain/registry, we're using add/update/etc for create/update actions. We should be consistent in kn-admin.
kn client has a story to use create/apply/remove. We can follow the same way to keep same user experience.
The text was updated successfully, but these errors were encountered: