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 aliases to --batch global flag and #77

Merged
merged 1 commit into from
Sep 28, 2022
Merged

Add aliases to --batch global flag and #77

merged 1 commit into from
Sep 28, 2022

Conversation

JOJ0
Copy link
Owner

@JOJ0 JOJ0 commented Sep 24, 2022

describe more precicesly what it does.

describe more precicesly what it does.
@JOJ0 JOJ0 requested a review from Ascurius September 24, 2022 08:17
@JOJ0
Copy link
Owner Author

JOJ0 commented Sep 24, 2022

@a-0-dev, since we were talking about this option, your review kindly requested! Thanks :-)

@a-0-dev
Copy link

a-0-dev commented Sep 26, 2022

Yep looks like what we were talking about. Some tools (like apt-get) also use -y as shorthand for --yes, if there's no better use for that shorthand from your perspective I would suggest including that as well

@JOJ0
Copy link
Owner Author

JOJ0 commented Sep 26, 2022

I do agree that adding --yes to the list is fine and as you put it, nice because other tools use it, it's quite common!

But if we are being picky we could argue that switching off translation of unix timestamps to human readable is not what people would expect from a --yes option...

--batch(-processing) and --scripting are better fits actually.

@JOJ0 JOJ0 merged commit 9f82ad8 into master Sep 28, 2022
@JOJ0
Copy link
Owner Author

JOJ0 commented Sep 28, 2022

Anyway, I merge this now since it's documented clearly that --yes does a little more than saying "yes" ;-) It's documented, it's fine :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants