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

Improve command-line usage #23

Merged
merged 4 commits into from
May 13, 2024
Merged

Improve command-line usage #23

merged 4 commits into from
May 13, 2024

Conversation

sveinse
Copy link
Member

@sveinse sveinse commented Apr 29, 2024

This PR improves command-line, with special attention on usage for listing ODs. The backing functions for generating the output have been improved.

* Add `--no-color`
* Remove `--header`
* Change `--nosort` to `--no-sort` for consistency
* Fixup help functions
* Fix `list -i` bug, not listing the specified indexes
* Removed Node.GetEntryFlag()
* Refactored and renamed Node.GetPrintLine() to Node.GetPrintEntryHeader()
* Refactored and renamed Node.GetPrintParams() to Node.GetPrintEntry()
* Fixed `--raw` usage
* Fixed `--unused` showing unused objects
* Improved colored output
* Renamed test file test_node.py to test_maps.py
* Add test_main and test_node
* Fix bugs in nodelist and eds_utils from testing
Copy link

sonarcloud bot commented May 4, 2024

Copy link

@svengetaz svengetaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No obvious issues as far as I can see.

@sveinse sveinse merged commit d798d6c into main May 13, 2024
3 checks passed
@sveinse sveinse deleted the feature-improve-cmdline branch May 13, 2024 07:11
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