Skip to content
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 -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 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants