| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
* Drop support for Python 3.7
* Update py-version + classifier
* Update functional tests
|
| |
|
| |
|
| |
|
|
|
|
| |
refs #8509
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
| |
|
|
|
|
|
| |
* Remove all old config parsing code
* Temporarily disable a test
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
| |
* Changing flake8 max-line-length to 120
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
| |
Co-authored-by: Andreas Finkler <3929834+DudeNr33@users.noreply.github.com>
|
|
|
| |
Co-authored-by: Andreas Finkler <3929834+DudeNr33@users.noreply.github.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Lower max locals
* Lower max statements
* Lower max try statements
* Lower max branches
* Lower max args
* Cut design constraints with default values
|
|
|
| |
125 is a good start. The check was activated in pylint with value = 100, but flake8 is less lenient than pylint and does not make any exceptions (for docstrings, strings and comments in particular).
|
| |
|
|
|
|
|
|
|
|
| |
Ensure that the import path is fixed up before calling
._astroid_module_checker() so that the pylint_django plugin can
successfully import the Django settings module when its checkers are
initialized.
Closes #7938
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Replace multiprocessing.pool with concurrent.futures.ProcessPoolExecutor to avoid deadlocks.
In a multiprocessing.pool, if a process terminates in a non-clean fashion
(for example, due to OOM or a segmentation fault), the pool will silently
replace said process, but the work that the process was supposed to do
will never be done, causing pylint to hang indefinitely.
The concurrent.futures.ProcessPoolExecutor will raise a
BrokenProcessPool exception in that case, avoiding the hang.
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
| |
Use this flag if you expect the linted files to be altered between runs,
for instance, if using pylint in a server-like mode. The flag clear's
astroid's in-memory caches.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Use dict for 'expand_modules' result rather than list.
With 'path' as the key, we get deduplication for free
and do not need to reprocess the list for deduplication later.
* Fix deduplication to account for CLI args marker.
* Fix corner case with CLI arg flag handling during deduplication.
* Add 'deduplication' to custom Pyenchant dict.
Closes #6242
Closes #4053
Co-authored-by: Eric McDonald <emcd@users.noreply.github.com>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
| |
Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
|
|
|
|
|
| |
* Refactor MessageDefinitionStore to accept py_version
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
When the dictionary has served its purpose (making plugin loading
pre-and-post init a consistent behaviour), we swap it for bools
indicating whether or not a module was loaded.
We don't currently use the bools, but it seemed a sensible choice.
The main idea is to make the dictionary fully pickle-able, so that when
dill pickles the linter for multiprocessing, it doesn't crash horribly.
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
|
|
|
|
|
| |
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
| |
|
|
|
|
|
|
|
|
| |
sense. (#7465)
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
|
|
|
|
| |
* Ignore subclasses of Any
* Add ignores for imported code and remove casts
* Add disables for uninferable return values
* Turn on ``mypy`` strict mode
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add x-failing test for issue 7264 case 3
This is the case where a plugin can be imported only after the init-hook
is run, and the init hook is specified in a pylintrc file
We would expect the module to not load in any case, and cause the
emission of a bad-plugin-value message.
* _dynamic_plugins is a dict not a set
This is in preparation for caching the loaded modules, and for storing
other information that will help in identifying times loading is
dependent on init-hook magic.
* Use cached module objects for load_configuration
This fixes case 3 in #7264, and is technically a breaking change, in
that it now emits a message for what would have previously been a silent
failure.
* Interim review comment fixes
1. Remove the xfail that snuck back in after the rebases.
2. Now that fake_home yields a str, update the type hint.
* Add test for bad plugin with duplicated load
This is case 6 in issue #7264, and represents the other silent failure
that is being made non-silent by this change.
* Apply review feedback
- Add an ordering test for CLI arguments
- Add clarifying comments on affected functions
- Tidy up test assertions to be more pythonic
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
| |
|
|
|
|
|
|
| |
Previously pylint would lint a file passed on stdin even if the user
meant to ignore the file. This commit fixes that issue.
Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
|
| |
|
|
|
|
|
|
|
|
|
| |
* Set ``_ignore_paths`` in ``initiliaze()``
* Fix typing
* Extract FileItems before getting asts
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
|
|
|
|
|
|
|
|
|
| |
directory" (#7304)
This reverts commit 3ebb700aed88a427011c2e88705f66ec0b3830a4.
* Add regression test for namespace packages under directory on path
* Fix expected result in `test_relative_beyond_top_level_two`
|
| |
|