-
Notifications
You must be signed in to change notification settings - Fork 2
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 support for ruff #18
Conversation
f1926fc
to
2384dfa
Compare
""" | ||
Run `ruff check` and return a dict with its results | ||
""" | ||
cmd = ["ruff", "check", "--output-format=json"] + sys.argv[1:] |
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.
Should it be added as a weak deps to spec file?
Should the argv be sanitized?
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.
Should it be added as a weak deps to spec file?
It should, thank you. Good catch.
Should the argv be sanitized?
I don't know, I did the same that vcs-diff-lint-csdiff-pylint
does. But probably not? Worst case scenario, the bad values are going to be passed to ruff check
which will handle it and error out.
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.
Right, with the shell=False (that is the default) this is not a problem.
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.
+1
Thank you! |
Fix #17
See https://docs.astral.sh/ruff