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.

Sign up for GitHub

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

Jump to bottom

Nano's CLI --help should display options in alphabetic order #3748

Closed
ealtamir opened this issue Mar 7, 2022 · 1 comment · Fixed by #3749
Closed

Nano's CLI --help should display options in alphabetic order #3748

ealtamir opened this issue Mar 7, 2022 · 1 comment · Fixed by #3749
Labels
cli Changes related to the Command Line Interface
Milestone
V24.0

Comments

@ealtamir
Copy link

ealtamir commented Mar 7, 2022

Summary

Explained in the title

What problem would be solved by this feature?

The current order has poor usability making it harder to find what one is looking for.

Are there any previous requests for this feature?

I believe not.

Do you have a suggested solution?

Sort the options in alphabetic order.

If this feature is approved, would you be willing to submit a pull request with the solution?

I would not

Possible solution

No response

Supporting files

No response
@Justin-Randall
Copy link
Contributor

I can get a PR together for this. This is a nice touch for programs with lots of options (as nano_node does, for example).

Justin-Randall added a commit to Justin-Randall/nano-node that referenced this issue Mar 10, 2022
@Justin-Randall
Fix Issue nanocurrency#3748
85e15b3 
Sorted command line options output
@zhyatt zhyatt added this to the V24.0 milestone Mar 10, 2022
@zhyatt zhyatt added the cli Changes related to the Command Line Interface label Mar 10, 2022
@zhyatt zhyatt linked a pull request Mar 10, 2022 that will close this issue
Fix Issue #3748 #3749
Merged
clemahieu pushed a commit that referenced this issue Mar 10, 2022
@Justin-Randall
Fix Issue #3748 (#3749)
8db9b39 
* Fix Issue #3748
Sorted command line options output

* Missing link library failing tests
clemahieu pushed a commit that referenced this issue Mar 14, 2022
@Justin-Randall
Fix_fork_storm failure #3709 on node allocation and catch stalled vot…
4512cfd 
…e/fork resolution errorrs in the test (#3750)

* Fix Issue #3748
Sorted command line options output

* Missing link library failing tests

* Issue #3709 (WIP)
Reduce number of nodes spawned for reasonable limits on most systems.
C++ coding style and efficiency changes for iteration.
Add diagnostic message streaming as an example.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cli Changes related to the Command Line Interface
Projects
None yet
Milestone
V24.0
Development

Successfully merging a pull request may close this issue.

Fix Issue #3748 Justin-Randall/nano-node
3 participants
@ealtamir @zhyatt @Justin-Randall