Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add -s/--short option to ferium upgrade to hide compatible mods #323

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tippfehlr
Copy link

I have ferium upgrade in my upgrade script and printing all mods every time is unnecessary in my case. I only want to see mods that are actually upgraded.

@T3sT3ro
Copy link

T3sT3ro commented May 24, 2023

-l in this case is confusing for any linux/terminal user, as it's an almost standard for "long list format" or otherwise for "long output", the format that --verbose uses.

--less is an unfortunate naming for all linux users, as it's a standard tool for displaying files in terminal. There are many tools that use --less argument to use less as a pager and it's confusing.

A much better and standardized argument option is -q, --quiet, --silent which suppresses printing to terminal.

@tippfehlr
Copy link
Author

You're right, I didn't think about that (I'm a Linux user myself). Do you think -q/--quiet would be the right choice? The command line flag only suppresses the first part and doesn't print nothing.

@T3sT3ro
Copy link

T3sT3ro commented May 24, 2023

I think I would go with -s --short for the short output like yours and with -q --quiet for no output at all (exit code only).

@tippfehlr tippfehlr force-pushed the ferium-upgrade-less branch from 7c4b16f to 1371522 Compare May 24, 2023 14:48
@tippfehlr tippfehlr changed the title add -l/--less option to ferium upgrade to hide compatible mods add -s/--short option to ferium upgrade to hide compatible mods May 24, 2023
@tippfehlr tippfehlr mentioned this pull request Aug 18, 2023
11 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants