-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Document cli options that can be set in configuration #2693
Document cli options that can be set in configuration #2693
Conversation
Remark: regarding sort order of SectionAll configuration settings It looks like all these list should be sorted which imho is a good setup.
if you want I could add another commit to align the sort (or create another PR or leave as is) just as you prefer. |
Thanks. Please rebase and make the sort corrections you mentioned. |
092596e
to
56fd858
Compare
rebased, sorted, updated PR and set to Review |
Thx for the review - I just took the stuff literally from the comandline help just one suggestion: What about using the phrase
maybe without capitalizing warnings.
if you don't like that, I'll take your proposed changes for the four open ones as is |
Sure. |
At this point I really don't care. Let's just get it done. |
Fix sort order of options in section 'All configuration settings' add changes requested by review add changes requested by review as discussed
199b78e
to
a8b3571
Compare
hope that finally fits |
implements Forum Topic 51064