-
-
Notifications
You must be signed in to change notification settings - Fork 241
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
Add support for --column
to all list commands
#1027
Comments
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of #1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
I would like to continue working on this issue, can it be assigned to me? Thanks. |
Done! |
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
While going through them. I noticed that the long description is not consistent across all commands. For example:
VS
Also, some of them are misleading. In network list it says "control which image attributes" instead of "control which network attributes". I fell for that myself in some of the previous PRs. I can fix that indeed, but should I open PRs directly, or should an issue be open first for fixing the long descriptions? |
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Feel free to open PRs directly to fix those. I suspect we did a bunch of copy/pasting over the years :) |
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Okay. Thanks. |
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of #1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of #1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of #1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of #1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of #1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of #1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of #1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Part of lxc#1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Running
Is there a case that it returns multiple columns? If not, is column selection still needed here? |
@HassanAlsamahi yeah, we can skip that one. |
Part of #1027 Signed-off-by: HassanAlsamahi <hassanalsamahi12@gmail.com>
Splitting this one out of #1026.
The full list of commands that currently do not support a
--columns
option are:The text was updated successfully, but these errors were encountered: