-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Multiple output-format
support
#4492
Multiple output-format
support
#4492
Conversation
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.
Thank you for this merge request. I made some comment to address before merging but I also think we'll need more tests than that, I did not check in detail but I don't think a lot of test cases are tested right now.
@Pierre-Sassoulas done, I think. Regarding type hints, I've tried my best, but I may need some help with the following:
|
Well, there weren't a lot of tests for |
@Pierre-Sassoulas I think I've addressed all the requested changes and all the checks pass now. |
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.
Thank you for this new option and for adding tests in what was not previously covered. Congratulation on becoming a pylint contributor :)
Btw, while I have your attention. Do you know, what will the next |
Pylint 2.9.0 will release very soon (after astroid 2.5.7), pylint 3.0.0alpha is a version including breaking changes (See #3504) that is not going out of alpha before a long time, 3.0.0a4 will release at the same time than 2.9.0. |
Steps
doc/whatsnew/<current release.rst>
.Description
This PR allows users to specify more than one output format by passing a comma-separated list
output-format
like this:You can also set the output file for each output format by specifying it after a colon like this:
Without the
:path
part, the default is to output tostdout
.Type of Changes
Related Issue
Closes #1798