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
Analyzer: Remove python-click dependency #5913
Analyzer: Remove python-click dependency #5913
Conversation
src/tools/analyzer/sss_analyze.py
Outdated
""" | ||
|
||
class _PseudoGroup(argparse.Action): | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpicking - Looking at the overall style this empty line should be avoided.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, fixed - and rebased PR.
9ba686c
to
5e59fa9
Compare
Thank you for the patch. I will wait for CI to finish the job and then I'm going to ACK it. |
ACK |
from sss_analyze import SubparsersAction | ||
from sss_analyze import Option | ||
from sss_analyze import Analyzer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
incomplete imports
see #5842
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, thank you.
As python-click will not be in RHEL9, switch to using the builtin argparse python module.
5e59fa9
to
de3635a
Compare
@thalman, are you fine with the latest update? @stanislavlevin, does it address your comment? |
@alexey-tikhonov Yes, I'm fine with last update |
@alexey-tikhonov, yes, it does. |
As python-click will not be in RHEL9, switch to using the builtin argparse python module.