| Commit message (Collapse) | Author | Age | Files | Lines |
| | |
|
| |
|
|
|
|
| |
MsgStore.get_messages_from_checker => Checker.messages
MsgStore.check_checker_consistency => Checker.check_consistency
Probably makes more sense this way.
|
| |
|
|
|
|
|
|
| |
Some constants were package internal but were used by multiple
packages. This created circular dependencies. By creating a
file for constants we make sure this does not happen because
we won't import everything important in this file and every
thing else can depend on it.
|
| |
|
|
| |
Taken from unittest_lint.py
|
| |
|
|
|
|
|
|
| |
There is a lot of Message related class in Utils
this warrant the creation of a new package.
See also review for burst utils.py into a package here:
https://github.com/PyCQA/pylint/pull/2654#issuecomment-470748956
|
| |
|
|
|
|
|
| |
A new optional hook for plugins is added: `load_configuration()`.
This hook is executed after configuration is loaded to allow plugins
to provide a specific configuration tailored to their purpose (e.g.
pylint-django could set up their own set of `good_names` or blacklisted
directories)
|
| | |
|
| |
|
|
|
|
|
| |
We return a list of message definitions so we must rename
get_message_definition to get_message_definitions. Resulting variables
are now named message_definitions most of the time in order to improve
readability. (It was often named "msgs" or worst "symbol")
|
| |
|
|
| |
Of MessageDefinition instead of a MessageDefinition.
|
| |
|
|
| |
test for it in unittest_utils
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
Each ChildLinter receives a file path and instantiates a linter to check
that file. As such, all files are arguments to child linters in parallel
mode. Therefore, the check for should_analyze_file must happen in the
parent linter, where knowledge of the original arguments is available.
Expand the custom should_analyze_file test to exercise parallel mode.
Close #1885
|
| | |
|
| | |
|
| |
|
|
| |
Closes #2071
|
| |
|
|
| |
See #2075
|
| |
|
|
|
|
|
|
| |
Create a function to get message definitions list from a checker.
Create a function for checking checker consistency
Create a function in order to register a MessageDefinition
Corrected following the review of pull-request #2036 by Ashley Whetter and PCManticore.
|
| |
|
|
| |
Closes #1120
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Previously, `pylint.utils.expand_modules` identified `__init__.py` files by a
substring match, which led to false positives for files with names like
`flycheck__init__.py`. (While users are unlikely to choose such a filename,
tools like flycheck use these sorts of filenames for temporary files.)
The result was that `pylint` would end up linting the entire package, not just
the file in question.
The fix is straightforward -- we use `os.path.basename` instead of a substring
check, and add a test.
|
| | |
|
| |
|
| |
https://docs.python.org/3/whatsnew/3.6.html#deprecated-python-behavior
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
Close #1079
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
Add InvalidMessageError and use it instead of asserts in pylint.utils.
|
| |
|
|
|
| |
This was changed automatically in #894, but apparently
we need to have the copyright notice somewhere.
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
| |
The html reporter was lately a second class citizen in Pylint,
being underdeveloped and mostly neglected. It has a couple of bugs,
but the more important reason why this is removed is that we now have
a much better json reporter, which can be used as a basis for more
prettier html Pylint outputs.
Part of #975
|
| | |
|
| |
|
|
| |
with local namespace packages, such as no-name-in-module.
|
| |
|
|
|
|
|
|
|
|
|
| |
As per discussion from issue #746, the reports were disabled by
default in order to simplify the interaction between the
tool and the users. The score is still shown by default,
as a way of closely measuring when it increases or decreases
due to changes brought to the code.
The patch introduces a new command line flag, "--score" or its
shorthand version, "-s", which controls if the score is shown or not.
By default, it's set to true.
|
| |
|
|
|
|
|
|
|
|
| |
in plugin modules.
If a plugin uses an old visit method (visit_class for instance), this can lead
to a crash in pylint's base checkers, because the logic in the PylintASTWalker
assumes that all checkers have a visit_class / leave_class method. The
patch fixes this by looking for both names.
Closes issue #711.
|
| | |
|
| |
|
|
| |
required-attributes, is slated for removal in Pylint 1.6.
|
| |
|
|
|
| |
--HG--
branch : no-logilab-common
|
| |
|
|
|
| |
--HG--
branch : no-logilab-common
|