-
-
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
Stop checking minimum name length by default #8813
Conversation
This comment has been minimized.
This comment has been minimized.
This test result change is reproducible on main by changing `as b` to `as boo` so that it met the prior invalid-name criterion. I don't know what the source of the interaction is between these two messages.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #8813 +/- ##
=======================================
Coverage 95.89% 95.89%
=======================================
Files 173 173
Lines 18511 18511
=======================================
Hits 17751 17751
Misses 760 760
|
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.
Took some time to review because this is a complicated topic that was discussed a lot (the reason why nothing was done for a long time). For example I wonder if this closes #7305 ?
No rush! I think #7305 is still valid. That's the third option I alluded to, where we could add a new feature like I just didn't want to block this change on #7305 and in the meantime, until someone volunteers for #7305, then we at least have a simple, short regex-based workaround. |
Thank you for taking a decision to make pylint better now instead of waiting for a super clean design later @jacobtylerwalls, we were (I was) clearly in decision paralysis for too long. Amazing solution, as in amazingly simple and amazingly impactful ! |
This reverts commit fdd914a.
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [pylint](https://togithub.com/pylint-dev/pylint) ([changelog](https://pylint.readthedocs.io/en/latest/whatsnew/2/)) | `>=2.17.4,<2.18` -> `>=3,<3.1` | [![age](https://developer.mend.io/api/mc/badges/age/pypi/pylint/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/pylint/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/pylint/2.17.7/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/pylint/2.17.7/3.0.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>pylint-dev/pylint (pylint)</summary> ### [`v3.0.0`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.0.0) Pylint now support python 3.12 officially. This long anticipated major version also provides some important usability and performance improvements, along with enacting necessary breaking changes and long-announced deprecations. The documentation of each message with an example is very close too. The required astroid version is now 3.0.0. See the [astroid changelog](https://pylint.readthedocs.io/projects/astroid/en/latest/changelog.html#what-s-new-in-astroid-3-0-0) for additional fixes, features, and performance improvements applicable to pylint. Our code is now fully typed. The invalid-name message no longer checks for a minimum length of 3 characters by default. Dependencies like wrapt or setuptools were removed. A new json2 reporter has been added. It features an enriched output that is easier to parse and provides more info, here's a sample output. ```json { "messages": [ { "type": "convention", "symbol": "line-too-long", "message": "Line too long (1/2)", "messageId": "C0301", "confidence": "HIGH", "module": "0123", "obj": "", "line": 1, "column": 0, "endLine": 1, "endColumn": 4, "path": "0123", "absolutePath": "0123" } ], "statistics": { "messageTypeCount": { "fatal": 0, "error": 0, "warning": 0, "refactor": 0, "convention": 1, "info": 0 }, "modulesLinted": 1, "score": 5.0 } } ``` ## Breaking Changes - Enabling or disabling individual messages will now take effect even if an `--enable=all` or `disable=all` follows in the same configuration file (or on the command line). This means for the following example, `fixme` messages will now be emitted: ```bash pylint my_module --enable=fixme --disable=all ``` To regain the prior behavior, remove the superfluous earlier option. Closes [#​3696](https://togithub.com/pylint-dev/pylint/issues/3696) - Remove support for launching pylint with Python 3.7. Code that supports Python 3.7 can still be linted with the `--py-version=3.7` setting. Refs [#​6306](https://togithub.com/pylint-dev/pylint/issues/6306) - Disables placed in a `try` block now apply to the `except` block. Previously, they only happened to do so in the presence of an `else` clause. Refs [#​7767](https://togithub.com/pylint-dev/pylint/issues/7767) - `pyreverse` now uses a new default color palette that is more colorblind friendly. The color scheme is taken from `Paul Tol's Notes <https://personal.sron.nl/~pault/>`\_. If you prefer other colors, you can use the `--color-palette` option to specify custom colors. Closes [#​8251](https://togithub.com/pylint-dev/pylint/issues/8251) - Everything related to the `__implements__` construct was removed. It was based on PEP245 that was proposed in 2001 and rejected in 2006. The capability from pyreverse to take `__implements__` into account when generating diagrams was also removed. Refs [#​8404](https://togithub.com/pylint-dev/pylint/issues/8404) - `pyreverse`: Support for the `.vcg` output format (Visualization of Compiler Graphs) has been dropped. Closes [#​8416](https://togithub.com/pylint-dev/pylint/issues/8416) - The warning when the now useless old pylint cache directory (pylint.d) was found was removed. The cache dir is documented in `the FAQ <https://pylint.readthedocs.io/en/latest/faq.html#where-is-the-persistent-data-stored-to-compare-between-successive-runs>`\_. Refs [#​8462](https://togithub.com/pylint-dev/pylint/issues/8462) - Following a deprecation period, `pylint.config.PYLINTRC` was removed. Use the `pylint.config.find_default_config_files` generator instead. Closes [#​8862](https://togithub.com/pylint-dev/pylint/issues/8862) ## Changes requiring user actions - The `invalid-name` message no longer checks for a minimum length of 3 characters by default. (This was an unadvertised commingling of concerns between casing and name length, and users regularly reported this to be surprising.) If checking for a minimum length is still desired, it can be regained in two ways: - If you are content with a `disallowed-name` message (instead of `invalid-name`), then simply add the option `bad-names-rgxs="^..?$"`, which will fail 1-2 character-long names. (Ensure you enable `disallowed-name`.) - If you would prefer an `invalid-name` message to be emitted, or would prefer finer-grained control over the circumstances in which messages are emitted (classes vs. methods, etc.), then avail yourself of the regex options described `here <https://pylint.readthedocs.io/en/stable/user_guide/configuration/all-options.html#main-checker>`*. (In particular, take note of the commented out options in the "example configuration" given at the bottom of the section.) The prior regexes can be found in the `pull request <https://github.com/pylint-dev/pylint/pull/8813>`* that removed the length requirements. Closes [#​2018](https://togithub.com/pylint-dev/pylint/issues/2018) - The compare to empty string checker (`pylint.extensions.emptystring`) and the compare to zero checker (`pylint.extensions.compare-to-zero`) have been removed and their checks are now part of the implicit booleaness checker: - `compare-to-zero` was renamed `use-implicit-booleaness-not-comparison-to-zero` and `compare-to-empty-string` was renamed `use-implicit-booleaness-not-comparison-to-string` and they now need to be enabled explicitly. - The `pylint.extensions.emptystring` and `pylint.extensions.compare-to-zero` extensions no longer exist and need to be removed from the `load-plugins` option. - Messages related to implicit booleaness were made more explicit and actionable. This permits to make their likeness explicit and will provide better performance as they share most of their conditions to be raised. Closes [#​6871](https://togithub.com/pylint-dev/pylint/issues/6871) - epylint was removed. It now lives at: https://github.com/emacsorphanage/pylint. Refs [#​7737](https://togithub.com/pylint-dev/pylint/issues/7737) - The `overgeneral-exceptions` option now only takes fully qualified names into account (`builtins.Exception` not `Exception`). If you overrode this option, you need to use the fully qualified name now. There's still a warning, but it will be removed in 3.1.0. Refs [#​8411](https://togithub.com/pylint-dev/pylint/issues/8411) - Following a deprecation period, it's no longer possible to use `MASTER` or `master` as configuration section in `setup.cfg` or `tox.ini`. It's bad practice to not start a section title with the tool name. Please use `pylint.main` instead. Refs [#​8465](https://togithub.com/pylint-dev/pylint/issues/8465) - Package stats are now printed when running Pyreverse and a `--verbose` flag was added to get the original output with parsed modules. You might need to activate the verbose option if you want to keep the old output. Closes [#​8973](https://togithub.com/pylint-dev/pylint/issues/8973) ## New Features - A new `json2` reporter has been added. It features a more enriched output that is easier to parse and provides more info. Compared to `json` the only changes are that messages are now under the `"messages"` key and that `"message-id"` now follows the camelCase convention and is renamed to `"messageId"`. The new reporter also reports the "score" of the modules you linted as defined by the `evaluation` option and provides statistics about the modules you linted. We encourage users to use the new reporter as the `json` reporter will no longer be maintained. Closes [#​4741](https://togithub.com/pylint-dev/pylint/issues/4741) - In Pyreverse package dependency diagrams, show when a module imports another only for type-checking. Closes [#​8112](https://togithub.com/pylint-dev/pylint/issues/8112) - Add new option (`--show-stdlib`, `-L`) to `pyreverse`. This is similar to the behavior of `--show-builtin` in that standard library modules are now not included by default, and this option will include them. Closes [#​8181](https://togithub.com/pylint-dev/pylint/issues/8181) - Add Pyreverse option to exclude standalone nodes from diagrams with `--no-standalone`. Closes [#​8476](https://togithub.com/pylint-dev/pylint/issues/8476) ## New Checks - Added `DataclassChecker` module and `invalid-field-call` checker to check for invalid dataclasses.field() usage. Refs [#​5159](https://togithub.com/pylint-dev/pylint/issues/5159) - Add `return-in-finally` to emit a message if a return statement was found in a finally clause. Closes [#​8260](https://togithub.com/pylint-dev/pylint/issues/8260) - Add a new checker `kwarg-superseded-by-positional-arg` to warn when a function is called with a keyword argument which shares a name with a positional-only parameter and the function contains a keyword variadic parameter dictionary. It may be surprising behaviour when the keyword argument is added to the keyword variadic parameter dictionary. Closes [#​8558](https://togithub.com/pylint-dev/pylint/issues/8558) ## Extensions - Add new `prefer-typing-namedtuple` message to the `CodeStyleChecker` to suggest rewriting calls to `collections.namedtuple` as classes inheriting from `typing.NamedTuple` on Python 3.6+. Requires `load-plugins=pylint.extensions.code_style` and `enable=prefer-typing-namedtuple` to be raised. Closes [#​8660](https://togithub.com/pylint-dev/pylint/issues/8660) ## False Positives Fixed - Extend concept of "function ambiguity" in `safe_infer()` from differing number of function arguments to differing set of argument names. Solves false positives in `tensorflow`. Closes [#​3613](https://togithub.com/pylint-dev/pylint/issues/3613) - Fix `unused-argument` false positive when `__new__` does not use all the arguments of `__init__`. Closes [#​3670](https://togithub.com/pylint-dev/pylint/issues/3670) - Fix a false positive for `invalid-name` when a type-annotated class variable in an `enum.Enum` class has no assigned value. Refs [#​7402](https://togithub.com/pylint-dev/pylint/issues/7402) - Fix `unused-import` false positive for usage of `six.with_metaclass`. Closes [#​7506](https://togithub.com/pylint-dev/pylint/issues/7506) - Fix false negatives and false positives for `too-many-try-statements`, `too-complex`, and `too-many-branches` by correctly counting statements under a `try`. Refs [#​7767](https://togithub.com/pylint-dev/pylint/issues/7767) - When checking for unbalanced dict unpacking in for-loops, Pylint will now test whether the length of each value to be unpacked matches the number of unpacking targets. Previously, Pylint would test the number of values for the loop iteration, which would produce a false unbalanced-dict-unpacking warning. Closes [#​8156](https://togithub.com/pylint-dev/pylint/issues/8156) - Fix false positive for `used-before-assignment` when usage and assignment are guarded by the same test in different statements. Closes [#​8167](https://togithub.com/pylint-dev/pylint/issues/8167) - Adds `asyncSetUp` to the default `defining-attr-methods` list to silence `attribute-defined-outside-init` warning when using `unittest.IsolatedAsyncioTestCase`. Refs [#​8403](https://togithub.com/pylint-dev/pylint/issues/8403) - `logging-not-lazy` is not longer emitted for explicitly concatenated string arguments. Closes [#​8410](https://togithub.com/pylint-dev/pylint/issues/8410) - Fix false positive for isinstance-second-argument-not-valid-type when union types contains None. Closes [#​8424](https://togithub.com/pylint-dev/pylint/issues/8424) - `invalid-name` now allows for integers in `typealias` names: - now valid: `Good2Name`, `GoodName2`. - still invalid: `_1BadName`. Closes [#​8485](https://togithub.com/pylint-dev/pylint/issues/8485) - No longer consider `Union` as type annotation as type alias for naming checks. Closes [#​8487](https://togithub.com/pylint-dev/pylint/issues/8487) - `unnecessary-lambda` no longer warns on lambdas which use its parameters in their body (other than the final arguments), e.g. `lambda foo: (bar if foo else baz)(foo)`. Closes [#​8496](https://togithub.com/pylint-dev/pylint/issues/8496) - Fixed `unused-import` so that it observes the `dummy-variables-rgx` option. Closes [#​8500](https://togithub.com/pylint-dev/pylint/issues/8500) - `Union` typed variables without assignment are no longer treated as `TypeAlias`. Closes [#​8540](https://togithub.com/pylint-dev/pylint/issues/8540) - Allow parenthesized implicitly concatenated strings when `check-str-concat-over-line-jumps` is enabled. Closes [#​8552](https://togithub.com/pylint-dev/pylint/issues/8552). - Fix false positive for `positional-only-arguments-expected` when a function contains both a positional-only parameter that has a default value, and `**kwargs`. Closes [#​8555](https://togithub.com/pylint-dev/pylint/issues/8555) - Fix false positive for `keyword-arg-before-vararg` when a positional-only parameter with a default value precedes `*args`. Closes [#​8570](https://togithub.com/pylint-dev/pylint/issues/8570) - Fix false positive for `arguments-differ` when overriding `__init_subclass__`. Closes [#​8919](https://togithub.com/pylint-dev/pylint/issues/8919) - Fix a false positive for `no-value-for-parameter` when a staticmethod is called in a class body. Closes [#​9036](https://togithub.com/pylint-dev/pylint/issues/9036) ## False Negatives Fixed - Emit `used-before-assignment` when calling module-level functions before definition. Closes [#​1144](https://togithub.com/pylint-dev/pylint/issues/1144) - Apply `infer_kwarg_from_call()` to more checks These mostly solve false negatives for various checks, save for one false positive for `use-maxsplit-arg`. Closes [#​7761](https://togithub.com/pylint-dev/pylint/issues/7761) - `TypeAlias` variables defined in functions are now checked for `invalid-name` errors. Closes [#​8536](https://togithub.com/pylint-dev/pylint/issues/8536) - Fix false negative for `no-value-for-parameter` when a function, whose signature contains both a positional-only parameter `name` and also `*kwargs`, is called with a keyword-argument for `name`. Closes [#​8559](https://togithub.com/pylint-dev/pylint/issues/8559) - Fix a false negative for `too-many-arguments` by considering positional-only and keyword-only parameters. Closes [#​8667](https://togithub.com/pylint-dev/pylint/issues/8667) - Emit `assignment-from-no-return` for calls to builtin methods like `dict.update()`. Calls to `list.sort()` now raise `assignment-from-no-return` rather than `assignment-from-none` for consistency. Closes [#​8714](https://togithub.com/pylint-dev/pylint/issues/8714) Closes [#​8810](https://togithub.com/pylint-dev/pylint/issues/8810) - `consider-using-augmented-assign` is now applied to dicts and lists as well. Closes [#​8959](https://togithub.com/pylint-dev/pylint/issues/8959) ## Other Bug Fixes - Support `duplicate-code` message when parallelizing with `--jobs`. Closes [#​374](https://togithub.com/pylint-dev/pylint/issues/374) - Support `cyclic-import` message when parallelizing with `--jobs`. Closes [#​4171](https://togithub.com/pylint-dev/pylint/issues/4171) - `--jobs` can now be used with `--load-plugins`. This had regressed in astroid 2.5.0. Closes [#​4874](https://togithub.com/pylint-dev/pylint/issues/4874) - docparams extension considers type comments as type documentation. Closes [#​6287](https://togithub.com/pylint-dev/pylint/issues/6287) - When parsing comma-separated lists of regular expressions in the config, ignore commas that are inside braces since those indicate quantifiers, not delineation between expressions. Closes [#​7229](https://togithub.com/pylint-dev/pylint/issues/7229) - The `ignored-modules` option will now be correctly taken into account for `no-name-in-module`. Closes [#​7578](https://togithub.com/pylint-dev/pylint/issues/7578) - `sys.argv` is now always correctly considered as impossible to infer (instead of using the actual values given to pylint). Closes [#​7710](https://togithub.com/pylint-dev/pylint/issues/7710) - Avoid duplicative warnings for unqualified exception names in the `overgeneral-exceptions` setting when running with `--jobs`. Closes [#​7774](https://togithub.com/pylint-dev/pylint/issues/7774) - Don't show class fields more than once in Pyreverse diagrams. Closes [#​8189](https://togithub.com/pylint-dev/pylint/issues/8189) - Fix `used-before-assignment` false negative when TYPE_CHECKING imports are used in multiple scopes. Closes [#​8198](https://togithub.com/pylint-dev/pylint/issues/8198) - `--clear-cache-post-run` now also clears LRU caches for pylint utilities holding references to AST nodes. Closes [#​8361](https://togithub.com/pylint-dev/pylint/issues/8361) - Fix a crash when `TYPE_CHECKING` is used without importing it. Closes [#​8434](https://togithub.com/pylint-dev/pylint/issues/8434) - Fix a `used-before-assignment` false positive when imports are made under the `TYPE_CHECKING` else if branch. Closes [#​8437](https://togithub.com/pylint-dev/pylint/issues/8437) - Fix a regression of `preferred-modules` where a partial match was used instead of the required full match. Closes [#​8453](https://togithub.com/pylint-dev/pylint/issues/8453) - Fix a crash in pyreverse when "/" characters are used in the output filename e.g pyreverse -o png -p name/ path/to/project. Closes [#​8504](https://togithub.com/pylint-dev/pylint/issues/8504) - Don't show arrows more than once in Pyreverse diagrams. Closes [#​8522](https://togithub.com/pylint-dev/pylint/issues/8522) - Improve output of `consider-using-generator` message for `min()` calls with `default` keyword. Closes [#​8563](https://togithub.com/pylint-dev/pylint/issues/8563) - Fixed a crash when generating a configuration file: `tomlkit.exceptions.TOMLKitError: Can't add a table to a dotted key` caused by tomlkit `v0.11.8`. Closes [#​8632](https://togithub.com/pylint-dev/pylint/issues/8632) - Fix a line break error in Pyreverse dot output. Closes [#​8671](https://togithub.com/pylint-dev/pylint/issues/8671) - Fix a false positive for `method-hidden` when using `cached_property` decorator. Closes [#​8753](https://togithub.com/pylint-dev/pylint/issues/8753) - Dunder methods defined in lambda do not trigger `unnecessary-dunder-call` anymore, if they cannot be replaced by the non-dunder call. Closes [#​8769](https://togithub.com/pylint-dev/pylint/issues/8769) - Don't show duplicate type annotations in Pyreverse diagrams. Closes [#​8888](https://togithub.com/pylint-dev/pylint/issues/8888) - Fixing inconsistent hashing issue in `BaseChecker` causing some reports not being exported. Closes [#​9001](https://togithub.com/pylint-dev/pylint/issues/9001) - Don't add `Optional` to `|` annotations with `None` in Pyreverse diagrams. Closes [#​9014](https://togithub.com/pylint-dev/pylint/issues/9014) - Pyreverse doesn't show multiple class association arrows anymore, but only the strongest one. Refs [#​9045](https://togithub.com/pylint-dev/pylint/issues/9045) - Prevented data loss in the linter stats for messages relating to the linter itself (e.g. `unknown-option-value`), fixing problems with score, fail-on, etc. Closes [#​9059](https://togithub.com/pylint-dev/pylint/issues/9059) - Fix crash in refactoring checker when unary operand used with variable in for loop. Closes [#​9074](https://togithub.com/pylint-dev/pylint/issues/9074) ## Other Changes - Pylint now exposes its type annotations. Closes [#​5488](https://togithub.com/pylint-dev/pylint/issues/5488) and [#​2079](https://togithub.com/pylint-dev/pylint/issues/2079) - Search for `pyproject.toml` recursively in parent directories up to a project or file system root. Refs [#​7163](https://togithub.com/pylint-dev/pylint/issues/7163), Closes [#​3289](https://togithub.com/pylint-dev/pylint/issues/3289) - All code related to the optparse config parsing has been removed. Refs [#​8405](https://togithub.com/pylint-dev/pylint/issues/8405) - Pylint now supports python 3.12. Refs [#​8718](https://togithub.com/pylint-dev/pylint/issues/8718) - Add a CITATION.cff file to the root of the repository containing the necessary metadata to cite Pylint. Closes [#​8760](https://togithub.com/pylint-dev/pylint/issues/8760) - Renamed the "unneeded-not" error into "unnecessary_negation" to be clearer. Closes [#​8789](https://togithub.com/pylint-dev/pylint/issues/8789) ## Internal Changes - `get_message_definition` was removed from the base checker API. You can access message definitions through the `MessageStore`. Refs [#​8401](https://togithub.com/pylint-dev/pylint/issues/8401) - Everything related to the `__implements__` construct was removed. It was based on PEP245 that was proposed in 2001 and rejected in 2006. All the classes inheriting `Interface` in `pylint.interfaces` were removed. `Checker` should only inherit `BaseChecker` or any of the other checker types from `pylint.checkers`. `Reporter` should only inherit `BaseReporter`. Refs [#​8404](https://togithub.com/pylint-dev/pylint/issues/8404) - `modname` and `msg_store` are now required to be given in `FileState`. `collect_block_lines` has also been removed. `Pylinter.current_name` cannot be null anymore. Refs [#​8407](https://togithub.com/pylint-dev/pylint/issues/8407) - `Reporter.set_output` was removed in favor of `reporter.out = stream`. Refs [#​8408](https://togithub.com/pylint-dev/pylint/issues/8408) - A number of old utility functions and classes have been removed: `MapReduceMixin`: To make a checker reduce map data simply implement `get_map_data` and `reduce_map_data`. `is_inside_lambda`: Use `utils.get_node_first_ancestor_of_type(x, nodes.Lambda)` `check_messages`: Use `utils.only_required_for_messages` `is_class_subscriptable_pep585_with_postponed_evaluation_enabled`: Use `is_postponed_evaluation_enabled(node)` and `is_node_in_type_annotation_context(node)` `get_python_path`: assumption that there's always an **init**.py is not true since python 3.3 and is causing problems, particularly with PEP 420. Use `discover_package_path` and pass source root(s). `fix_import_path`: Use `augmented_sys_path` and pass additional `sys.path` entries as an argument obtained from `discover_package_path`. `get_global_option`: Use `checker.linter.config` to get all global options. Related private objects have been removed as well. Refs [#​8409](https://togithub.com/pylint-dev/pylint/issues/8409) - `colorize_ansi` now only accepts a `MessageStyle` object. Refs [#​8412](https://togithub.com/pylint-dev/pylint/issues/8412) - Following a deprecation period, `Pylinter.check` now only works with sequences of strings, not strings. Refs [#​8463](https://togithub.com/pylint-dev/pylint/issues/8463) - Following a deprecation period, `ColorizedTextReporter` only accepts `ColorMappingDict`. Refs [#​8464](https://togithub.com/pylint-dev/pylint/issues/8464) - Following a deprecation period, `MessageTest`'s `end_line` and `end_col_offset` must be accurate in functional tests (for python 3.8 or above on cpython, and for python 3.9 or superior on pypy). Refs [#​8466](https://togithub.com/pylint-dev/pylint/issues/8466) - Following a deprecation period, the `do_exit` argument of the `Run` class (and of the `_Run` class in testutils) were removed. Refs [#​8472](https://togithub.com/pylint-dev/pylint/issues/8472) - Following a deprecation period, the `py_version` argument of the `MessageDefinition.may_be_emitted` function is now required. The most likely solution is to use 'linter.config.py_version' if you need to keep using this function, or to use 'MessageDefinition.is_message_enabled' instead. Refs [#​8473](https://togithub.com/pylint-dev/pylint/issues/8473) - Following a deprecation period, the `OutputLine` class now requires the right number of argument all the time. The functional output can be regenerated automatically to achieve that easily. Refs [#​8474](https://togithub.com/pylint-dev/pylint/issues/8474) - Following a deprecation period, `is_typing_guard`, `is_node_in_typing_guarded_import_block` and `is_node_in_guarded_import_block` from `pylint.utils` were removed: use a combination of `is_sys_guard` and `in_type_checking_block` instead. Refs [#​8475](https://togithub.com/pylint-dev/pylint/issues/8475) - Following a deprecation period, the `location` argument of the `Message` class must now be a `MessageLocationTuple`. Refs [#​8477](https://togithub.com/pylint-dev/pylint/issues/8477) - Following a deprecation period, the `check_single_file` function of the `Pylinter` is replaced by `Pylinter.check_single_file_item`. Refs [#​8478](https://togithub.com/pylint-dev/pylint/issues/8478) ## Performance Improvements - `pylint` runs (at least) ~5% faster after improvements to `astroid` that make better use of the inference cache. Refs [pylint-dev/astroid#529](https://togithub.com/pylint-dev/astroid/issues/529) - - Optimize `is_trailing_comma()`. - Cache `class_is_abstract()`. Refs [#​1954](https://togithub.com/pylint-dev/pylint/issues/1954) - Exit immediately if all messages are disabled. Closes [#​8715](https://togithub.com/pylint-dev/pylint/issues/8715) </details> --- ### Configuration π **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). π¦ **Automerge**: Disabled by config. Please merge this manually once you are satisfied. β» **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. π **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/hetznercloud/hcloud-python). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: jo <[email protected]>
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [pylint](https://togithub.com/pylint-dev/pylint) ([changelog](https://pylint.readthedocs.io/en/latest/whatsnew/2/)) | `==2.17.7` -> `==3.0.2` | [![age](https://developer.mend.io/api/mc/badges/age/pypi/pylint/3.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/pylint/3.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/pylint/2.17.7/3.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/pylint/2.17.7/3.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>pylint-dev/pylint (pylint)</summary> ### [`v3.0.2`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.0.2) [Compare Source](https://togithub.com/pylint-dev/pylint/compare/v3.0.1...v3.0.2) ## False Positives Fixed - Fix `used-before-assignment` false positive for generic type syntax (PEP 695, Python 3.12). Closes [#​9110](https://togithub.com/pylint-dev/pylint/issues/9110) ## Other Bug Fixes - Escape special symbols and newlines in messages. Closes [#​7874](https://togithub.com/pylint-dev/pylint/issues/7874) - Fixes suggestion for `nested-min-max` for expressions with additive operators, list and dict comprehensions. Closes [#​8524](https://togithub.com/pylint-dev/pylint/issues/8524) - Fixes ignoring conditional imports with `ignore-imports=y`. Closes [#​8914](https://togithub.com/pylint-dev/pylint/issues/8914) - Emit `inconsistent-quotes` for f-strings with 3.12 interpreter only if targeting pre-3.12 versions. Closes [#​9113](https://togithub.com/pylint-dev/pylint/issues/9113) ### [`v3.0.1`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.0.1) ## False Positives Fixed - Fixed false positive for `inherit-non-class` for generic Protocols. Closes [#​9106](https://togithub.com/pylint-dev/pylint/issues/9106) ## Other Changes - Fix a crash when an enum class which is also decorated with a `dataclasses.dataclass` decorator is defined. Closes [#​9100](https://togithub.com/pylint-dev/pylint/issues/9100) ### [`v3.0.0`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.0.0) Pylint now support python 3.12 officially. This long anticipated major version also provides some important usability and performance improvements, along with enacting necessary breaking changes and long-announced deprecations. The documentation of each message with an example is very close too. The required astroid version is now 3.0.0. See the [astroid changelog](https://pylint.readthedocs.io/projects/astroid/en/latest/changelog.html#what-s-new-in-astroid-3-0-0) for additional fixes, features, and performance improvements applicable to pylint. Our code is now fully typed. The invalid-name message no longer checks for a minimum length of 3 characters by default. Dependencies like wrapt or setuptools were removed. A new json2 reporter has been added. It features an enriched output that is easier to parse and provides more info, here's a sample output. ```json { "messages": [ { "type": "convention", "symbol": "line-too-long", "message": "Line too long (1/2)", "messageId": "C0301", "confidence": "HIGH", "module": "0123", "obj": "", "line": 1, "column": 0, "endLine": 1, "endColumn": 4, "path": "0123", "absolutePath": "0123" } ], "statistics": { "messageTypeCount": { "fatal": 0, "error": 0, "warning": 0, "refactor": 0, "convention": 1, "info": 0 }, "modulesLinted": 1, "score": 5.0 } } ``` ## Breaking Changes - Enabling or disabling individual messages will now take effect even if an `--enable=all` or `disable=all` follows in the same configuration file (or on the command line). This means for the following example, `fixme` messages will now be emitted: ```bash pylint my_module --enable=fixme --disable=all ``` To regain the prior behavior, remove the superfluous earlier option. Closes [#​3696](https://togithub.com/pylint-dev/pylint/issues/3696) - Remove support for launching pylint with Python 3.7. Code that supports Python 3.7 can still be linted with the `--py-version=3.7` setting. Refs [#​6306](https://togithub.com/pylint-dev/pylint/issues/6306) - Disables placed in a `try` block now apply to the `except` block. Previously, they only happened to do so in the presence of an `else` clause. Refs [#​7767](https://togithub.com/pylint-dev/pylint/issues/7767) - `pyreverse` now uses a new default color palette that is more colorblind friendly. The color scheme is taken from `Paul Tol's Notes <https://personal.sron.nl/~pault/>`\_. If you prefer other colors, you can use the `--color-palette` option to specify custom colors. Closes [#​8251](https://togithub.com/pylint-dev/pylint/issues/8251) - Everything related to the `__implements__` construct was removed. It was based on PEP245 that was proposed in 2001 and rejected in 2006. The capability from pyreverse to take `__implements__` into account when generating diagrams was also removed. Refs [#​8404](https://togithub.com/pylint-dev/pylint/issues/8404) - `pyreverse`: Support for the `.vcg` output format (Visualization of Compiler Graphs) has been dropped. Closes [#​8416](https://togithub.com/pylint-dev/pylint/issues/8416) - The warning when the now useless old pylint cache directory (pylint.d) was found was removed. The cache dir is documented in `the FAQ <https://pylint.readthedocs.io/en/latest/faq.html#where-is-the-persistent-data-stored-to-compare-between-successive-runs>`\_. Refs [#​8462](https://togithub.com/pylint-dev/pylint/issues/8462) - Following a deprecation period, `pylint.config.PYLINTRC` was removed. Use the `pylint.config.find_default_config_files` generator instead. Closes [#​8862](https://togithub.com/pylint-dev/pylint/issues/8862) ## Changes requiring user actions - The `invalid-name` message no longer checks for a minimum length of 3 characters by default. (This was an unadvertised commingling of concerns between casing and name length, and users regularly reported this to be surprising.) If checking for a minimum length is still desired, it can be regained in two ways: - If you are content with a `disallowed-name` message (instead of `invalid-name`), then simply add the option `bad-names-rgxs="^..?$"`, which will fail 1-2 character-long names. (Ensure you enable `disallowed-name`.) - If you would prefer an `invalid-name` message to be emitted, or would prefer finer-grained control over the circumstances in which messages are emitted (classes vs. methods, etc.), then avail yourself of the regex options described `here <https://pylint.readthedocs.io/en/stable/user_guide/configuration/all-options.html#main-checker>`*. (In particular, take note of the commented out options in the "example configuration" given at the bottom of the section.) The prior regexes can be found in the `pull request <https://github.com/pylint-dev/pylint/pull/8813>`* that removed the length requirements. Closes [#​2018](https://togithub.com/pylint-dev/pylint/issues/2018) - The compare to empty string checker (`pylint.extensions.emptystring`) and the compare to zero checker (`pylint.extensions.compare-to-zero`) have been removed and their checks are now part of the implicit booleaness checker: - `compare-to-zero` was renamed `use-implicit-booleaness-not-comparison-to-zero` and `compare-to-empty-string` was renamed `use-implicit-booleaness-not-comparison-to-string` and they now need to be enabled explicitly. - The `pylint.extensions.emptystring` and `pylint.extensions.compare-to-zero` extensions no longer exist and need to be removed from the `load-plugins` option. - Messages related to implicit booleaness were made more explicit and actionable. This permits to make their likeness explicit and will provide better performance as they share most of their conditions to be raised. Closes [#​6871](https://togithub.com/pylint-dev/pylint/issues/6871) - epylint was removed. It now lives at: https://github.com/emacsorphanage/pylint. Refs [#​7737](https://togithub.com/pylint-dev/pylint/issues/7737) - The `overgeneral-exceptions` option now only takes fully qualified names into account (`builtins.Exception` not `Exception`). If you overrode this option, you need to use the fully qualified name now. There's still a warning, but it will be removed in 3.1.0. Refs [#​8411](https://togithub.com/pylint-dev/pylint/issues/8411) - Following a deprecation period, it's no longer possible to use `MASTER` or `master` as configuration section in `setup.cfg` or `tox.ini`. It's bad practice to not start a section title with the tool name. Please use `pylint.main` instead. Refs [#​8465](https://togithub.com/pylint-dev/pylint/issues/8465) - Package stats are now printed when running Pyreverse and a `--verbose` flag was added to get the original output with parsed modules. You might need to activate the verbose option if you want to keep the old output. Closes [#​8973](https://togithub.com/pylint-dev/pylint/issues/8973) ## New Features - A new `json2` reporter has been added. It features a more enriched output that is easier to parse and provides more info. Compared to `json` the only changes are that messages are now under the `"messages"` key and that `"message-id"` now follows the camelCase convention and is renamed to `"messageId"`. The new reporter also reports the "score" of the modules you linted as defined by the `evaluation` option and provides statistics about the modules you linted. We encourage users to use the new reporter as the `json` reporter will no longer be maintained. Closes [#​4741](https://togithub.com/pylint-dev/pylint/issues/4741) - In Pyreverse package dependency diagrams, show when a module imports another only for type-checking. Closes [#​8112](https://togithub.com/pylint-dev/pylint/issues/8112) - Add new option (`--show-stdlib`, `-L`) to `pyreverse`. This is similar to the behavior of `--show-builtin` in that standard library modules are now not included by default, and this option will include them. Closes [#​8181](https://togithub.com/pylint-dev/pylint/issues/8181) - Add Pyreverse option to exclude standalone nodes from diagrams with `--no-standalone`. Closes [#​8476](https://togithub.com/pylint-dev/pylint/issues/8476) ## New Checks - Added `DataclassChecker` module and `invalid-field-call` checker to check for invalid dataclasses.field() usage. Refs [#​5159](https://togithub.com/pylint-dev/pylint/issues/5159) - Add `return-in-finally` to emit a message if a return statement was found in a finally clause. Closes [#​8260](https://togithub.com/pylint-dev/pylint/issues/8260) - Add a new checker `kwarg-superseded-by-positional-arg` to warn when a function is called with a keyword argument which shares a name with a positional-only parameter and the function contains a keyword variadic parameter dictionary. It may be surprising behaviour when the keyword argument is added to the keyword variadic parameter dictionary. Closes [#​8558](https://togithub.com/pylint-dev/pylint/issues/8558) ## Extensions - Add new `prefer-typing-namedtuple` message to the `CodeStyleChecker` to suggest rewriting calls to `collections.namedtuple` as classes inheriting from `typing.NamedTuple` on Python 3.6+. Requires `load-plugins=pylint.extensions.code_style` and `enable=prefer-typing-namedtuple` to be raised. Closes [#​8660](https://togithub.com/pylint-dev/pylint/issues/8660) ## False Positives Fixed - Extend concept of "function ambiguity" in `safe_infer()` from differing number of function arguments to differing set of argument names. Solves false positives in `tensorflow`. Closes [#​3613](https://togithub.com/pylint-dev/pylint/issues/3613) - Fix `unused-argument` false positive when `__new__` does not use all the arguments of `__init__`. Closes [#​3670](https://togithub.com/pylint-dev/pylint/issues/3670) - Fix a false positive for `invalid-name` when a type-annotated class variable in an `enum.Enum` class has no assigned value. Refs [#​7402](https://togithub.com/pylint-dev/pylint/issues/7402) - Fix `unused-import` false positive for usage of `six.with_metaclass`. Closes [#​7506](https://togithub.com/pylint-dev/pylint/issues/7506) - Fix false negatives and false positives for `too-many-try-statements`, `too-complex`, and `too-many-branches` by correctly counting statements under a `try`. Refs [#​7767](https://togithub.com/pylint-dev/pylint/issues/7767) - When checking for unbalanced dict unpacking in for-loops, Pylint will now test whether the length of each value to be unpacked matches the number of unpacking targets. Previously, Pylint would test the number of values for the loop iteration, which would produce a false unbalanced-dict-unpacking warning. Closes [#​8156](https://togithub.com/pylint-dev/pylint/issues/8156) - Fix false positive for `used-before-assignment` when usage and assignment are guarded by the same test in different statements. Closes [#​8167](https://togithub.com/pylint-dev/pylint/issues/8167) - Adds `asyncSetUp` to the default `defining-attr-methods` list to silence `attribute-defined-outside-init` warning when using `unittest.IsolatedAsyncioTestCase`. Refs [#​8403](https://togithub.com/pylint-dev/pylint/issues/8403) - `logging-not-lazy` is not longer emitted for explicitly concatenated string arguments. Closes [#​8410](https://togithub.com/pylint-dev/pylint/issues/8410) - Fix false positive for isinstance-second-argument-not-valid-type when union types contains None. Closes [#​8424](https://togithub.com/pylint-dev/pylint/issues/8424) - `invalid-name` now allows for integers in `typealias` names: - now valid: `Good2Name`, `GoodName2`. - still invalid: `_1BadName`. Closes [#​8485](https://togithub.com/pylint-dev/pylint/issues/8485) - No longer consider `Union` as type annotation as type alias for naming checks. Closes [#​8487](https://togithub.com/pylint-dev/pylint/issues/8487) - `unnecessary-lambda` no longer warns on lambdas which use its parameters in their body (other than the final arguments), e.g. `lambda foo: (bar if foo else baz)(foo)`. Closes [#​8496](https://togithub.com/pylint-dev/pylint/issues/8496) - Fixed `unused-import` so that it observes the `dummy-variables-rgx` option. Closes [#​8500](https://togithub.com/pylint-dev/pylint/issues/8500) - `Union` typed variables without assignment are no longer treated as `TypeAlias`. Closes [#​8540](https://togithub.com/pylint-dev/pylint/issues/8540) - Allow parenthesized implicitly concatenated strings when `check-str-concat-over-line-jumps` is enabled. Closes [#​8552](https://togithub.com/pylint-dev/pylint/issues/8552). - Fix false positive for `positional-only-arguments-expected` when a function contains both a positional-only parameter that has a default value, and `**kwargs`. Closes [#​8555](https://togithub.com/pylint-dev/pylint/issues/8555) - Fix false positive for `keyword-arg-before-vararg` when a positional-only parameter with a default value precedes `*args`. Closes [#​8570](https://togithub.com/pylint-dev/pylint/issues/8570) - Fix false positive for `arguments-differ` when overriding `__init_subclass__`. Closes [#​8919](https://togithub.com/pylint-dev/pylint/issues/8919) - Fix a false positive for `no-value-for-parameter` when a staticmethod is called in a class body. Closes [#​9036](https://togithub.com/pylint-dev/pylint/issues/9036) ## False Negatives Fixed - Emit `used-before-assignment` when calling module-level functions before definition. Closes [#​1144](https://togithub.com/pylint-dev/pylint/issues/1144) - Apply `infer_kwarg_from_call()` to more checks These mostly solve false negatives for various checks, save for one false positive for `use-maxsplit-arg`. Closes [#​7761](https://togithub.com/pylint-dev/pylint/issues/7761) - `TypeAlias` variables defined in functions are now checked for `invalid-name` errors. Closes [#​8536](https://togithub.com/pylint-dev/pylint/issues/8536) - Fix false negative for `no-value-for-parameter` when a function, whose signature contains both a positional-only parameter `name` and also `*kwargs`, is called with a keyword-argument for `name`. Closes [#​8559](https://togithub.com/pylint-dev/pylint/issues/8559) - Fix a false negative for `too-many-arguments` by considering positional-only and keyword-only parameters. Closes [#​8667](https://togithub.com/pylint-dev/pylint/issues/8667) - Emit `assignment-from-no-return` for calls to builtin methods like `dict.update()`. Calls to `list.sort()` now raise `assignment-from-no-return` rather than `assignment-from-none` for consistency. Closes [#​8714](https://togithub.com/pylint-dev/pylint/issues/8714) Closes [#​8810](https://togithub.com/pylint-dev/pylint/issues/8810) - `consider-using-augmented-assign` is now applied to dicts and lists as well. Closes [#​8959](https://togithub.com/pylint-dev/pylint/issues/8959) ## Other Bug Fixes - Support `duplicate-code` message when parallelizing with `--jobs`. Closes [#​374](https://togithub.com/pylint-dev/pylint/issues/374) - Support `cyclic-import` message when parallelizing with `--jobs`. Closes [#​4171](https://togithub.com/pylint-dev/pylint/issues/4171) - `--jobs` can now be used with `--load-plugins`. This had regressed in astroid 2.5.0. Closes [#​4874](https://togithub.com/pylint-dev/pylint/issues/4874) - docparams extension considers type comments as type documentation. Closes [#​6287](https://togithub.com/pylint-dev/pylint/issues/6287) - When parsing comma-separated lists of regular expressions in the config, ignore commas that are inside braces since those indicate quantifiers, not delineation between expressions. Closes [#​7229](https://togithub.com/pylint-dev/pylint/issues/7229) - The `ignored-modules` option will now be correctly taken into account for `no-name-in-module`. Closes [#​7578](https://togithub.com/pylint-dev/pylint/issues/7578) - `sys.argv` is now always correctly considered as impossible to infer (instead of using the actual values given to pylint). Closes [#​7710](https://togithub.com/pylint-dev/pylint/issues/7710) - Avoid duplicative warnings for unqualified exception names in the `overgeneral-exceptions` setting when running with `--jobs`. Closes [#​7774](https://togithub.com/pylint-dev/pylint/issues/7774) - Don't show class fields more than once in Pyreverse diagrams. Closes [#​8189](https://togithub.com/pylint-dev/pylint/issues/8189) - Fix `used-before-assignment` false negative when TYPE_CHECKING imports are used in multiple scopes. Closes [#​8198](https://togithub.com/pylint-dev/pylint/issues/8198) - `--clear-cache-post-run` now also clears LRU caches for pylint utilities holding references to AST nodes. Closes [#​8361](https://togithub.com/pylint-dev/pylint/issues/8361) - Fix a crash when `TYPE_CHECKING` is used without importing it. Closes [#​8434](https://togithub.com/pylint-dev/pylint/issues/8434) - Fix a `used-before-assignment` false positive when imports are made under the `TYPE_CHECKING` else if branch. Closes [#​8437](https://togithub.com/pylint-dev/pylint/issues/8437) - Fix a regression of `preferred-modules` where a partial match was used instead of the required full match. Closes [#​8453](https://togithub.com/pylint-dev/pylint/issues/8453) - Fix a crash in pyreverse when "/" characters are used in the output filename e.g pyreverse -o png -p name/ path/to/project. Closes [#​8504](https://togithub.com/pylint-dev/pylint/issues/8504) - Don't show arrows more than once in Pyreverse diagrams. Closes [#​8522](https://togithub.com/pylint-dev/pylint/issues/8522) - Improve output of `consider-using-generator` message for `min()` calls with `default` keyword. Closes [#​8563](https://togithub.com/pylint-dev/pylint/issues/8563) - Fixed a crash when generating a configuration file: `tomlkit.exceptions.TOMLKitError: Can't add a table to a dotted key` caused by tomlkit `v0.11.8`. Closes [#​8632](https://togithub.com/pylint-dev/pylint/issues/8632) - Fix a line break error in Pyreverse dot output. Closes [#​8671](https://togithub.com/pylint-dev/pylint/issues/8671) - Fix a false positive for `method-hidden` when using `cached_property` decorator. Closes [#​8753](https://togithub.com/pylint-dev/pylint/issues/8753) - Dunder methods defined in lambda do not trigger `unnecessary-dunder-call` anymore, if they cannot be replaced by the non-dunder call. Closes [#​8769](https://togithub.com/pylint-dev/pylint/issues/8769) - Don't show duplicate type annotations in Pyreverse diagrams. Closes [#​8888](https://togithub.com/pylint-dev/pylint/issues/8888) - Fixing inconsistent hashing issue in `BaseChecker` causing some reports not being exported. Closes [#​9001](https://togithub.com/pylint-dev/pylint/issues/9001) - Don't add `Optional` to `|` annotations with `None` in Pyreverse diagrams. Closes [#​9014](https://togithub.com/pylint-dev/pylint/issues/9014) - Pyreverse doesn't show multiple class association arrows anymore, but only the strongest one. Refs [#​9045](https://togithub.com/pylint-dev/pylint/issues/9045) - Prevented data loss in the linter stats for messages relating to the linter itself (e.g. `unknown-option-value`), fixing problems with score, fail-on, etc. Closes [#​9059](https://togithub.com/pylint-dev/pylint/issues/9059) - Fix crash in refactoring checker when unary operand used with variable in for loop. Closes [#​9074](https://togithub.com/pylint-dev/pylint/issues/9074) ## Other Changes - Pylint now exposes its type annotations. Closes [#​5488](https://togithub.com/pylint-dev/pylint/issues/5488) and [#​2079](https://togithub.com/pylint-dev/pylint/issues/2079) - Search for `pyproject.toml` recursively in parent directories up to a project or file system root. Refs [#​7163](https://togithub.com/pylint-dev/pylint/issues/7163), Closes [#​3289](https://togithub.com/pylint-dev/pylint/issues/3289) - All code related to the optparse config parsing has been removed. Refs [#​8405](https://togithub.com/pylint-dev/pylint/issues/8405) - Pylint now supports python 3.12. Refs [#​8718](https://togithub.com/pylint-dev/pylint/issues/8718) - Add a CITATION.cff file to the root of the repository containing the necessary metadata to cite Pylint. Closes [#​8760](https://togithub.com/pylint-dev/pylint/issues/8760) - Renamed the "unneeded-not" error into "unnecessary_negation" to be clearer. Closes [#​8789](https://togithub.com/pylint-dev/pylint/issues/8789) ## Internal Changes - `get_message_definition` was removed from the base checker API. You can access message definitions through the `MessageStore`. Refs [#​8401](https://togithub.com/pylint-dev/pylint/issues/8401) - Everything related to the `__implements__` construct was removed. It was based on PEP245 that was proposed in 2001 and rejected in 2006. All the classes inheriting `Interface` in `pylint.interfaces` were removed. `Checker` should only inherit `BaseChecker` or any of the other checker types from `pylint.checkers`. `Reporter` should only inherit `BaseReporter`. Refs [#​8404](https://togithub.com/pylint-dev/pylint/issues/8404) - `modname` and `msg_store` are now required to be given in `FileState`. `collect_block_lines` has also been removed. `Pylinter.current_name` cannot be null anymore. Refs [#​8407](https://togithub.com/pylint-dev/pylint/issues/8407) - `Reporter.set_output` was removed in favor of `reporter.out = stream`. Refs [#​8408](https://togithub.com/pylint-dev/pylint/issues/8408) - A number of old utility functions and classes have been removed: `MapReduceMixin`: To make a checker reduce map data simply implement `get_map_data` and `reduce_map_data`. `is_inside_lambda`: Use `utils.get_node_first_ancestor_of_type(x, nodes.Lambda)` `check_messages`: Use `utils.only_required_for_messages` `is_class_subscriptable_pep585_with_postponed_evaluation_enabled`: Use `is_postponed_evaluation_enabled(node)` and `is_node_in_type_annotation_context(node)` `get_python_path`: assumption that there's always an **init**.py is not true since python 3.3 and is causing problems, particularly with PEP 420. Use `discover_package_path` and pass source root(s). `fix_import_path`: Use `augmented_sys_path` and pass additional `sys.path` entries as an argument obtained from `discover_package_path`. `get_global_option`: Use `checker.linter.config` to get all global options. Related private objects have been removed as well. Refs [#​8409](https://togithub.com/pylint-dev/pylint/issues/8409) - `colorize_ansi` now only accepts a `MessageStyle` object. Refs [#​8412](https://togithub.com/pylint-dev/pylint/issues/8412) - Following a deprecation period, `Pylinter.check` now only works with sequences of strings, not strings. Refs [#​8463](https://togithub.com/pylint-dev/pylint/issues/8463) - Following a deprecation period, `ColorizedTextReporter` only accepts `ColorMappingDict`. Refs [#​8464](https://togithub.com/pylint-dev/pylint/issues/8464) - Following a deprecation period, `MessageTest`'s `end_line` and `end_col_offset` must be accurate in functional tests (for python 3.8 or above on cpython, and for python 3.9 or superior on pypy). Refs [#​8466](https://togithub.com/pylint-dev/pylint/issues/8466) - Following a deprecation period, the `do_exit` argument of the `Run` class (and of the `_Run` class in testutils) were removed. Refs [#​8472](https://togithub.com/pylint-dev/pylint/issues/8472) - Following a deprecation period, the `py_version` argument of the `MessageDefinition.may_be_emitted` function is now required. The most likely solution is to use 'linter.config.py_version' if you need to keep using this function, or to use 'MessageDefinition.is_message_enabled' instead. Refs [#​8473](https://togithub.com/pylint-dev/pylint/issues/8473) - Following a deprecation period, the `OutputLine` class now requires the right number of argument all the time. The functional output can be regenerated automatically to achieve that easily. Refs [#​8474](https://togithub.com/pylint-dev/pylint/issues/8474) - Following a deprecation period, `is_typing_guard`, `is_node_in_typing_guarded_import_block` and `is_node_in_guarded_import_block` from `pylint.utils` were removed: use a combination of `is_sys_guard` and `in_type_checking_block` instead. Refs [#​8475](https://togithub.com/pylint-dev/pylint/issues/8475) - Following a deprecation period, the `location` argument of the `Message` class must now be a `MessageLocationTuple`. Refs [#​8477](https://togithub.com/pylint-dev/pylint/issues/8477) - Following a deprecation period, the `check_single_file` function of the `Pylinter` is replaced by `Pylinter.check_single_file_item`. Refs [#​8478](https://togithub.com/pylint-dev/pylint/issues/8478) ## Performance Improvements - `pylint` runs (at least) ~5% faster after improvements to `astroid` that make better use of the inference cache. Refs [pylint-dev/astroid#529](https://togithub.com/pylint-dev/astroid/issues/529) - - Optimize `is_trailing_comma()`. - Cache `class_is_abstract()`. Refs [#​1954](https://togithub.com/pylint-dev/pylint/issues/1954) - Exit immediately if all messages are disabled. Closes [#​8715](https://togithub.com/pylint-dev/pylint/issues/8715) </details> --- ### Configuration π **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). π¦ **Automerge**: Disabled by config. Please merge this manually once you are satisfied. β» **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. π **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/iterative/gto). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4zMS41IiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [pylint](https://togithub.com/pylint-dev/pylint) ([changelog](https://pylint.readthedocs.io/en/latest/whatsnew/3/)) | `^2.15.10` -> `^3.0.0` | [![age](https://developer.mend.io/api/mc/badges/age/pypi/pylint/3.2.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/pylint/3.2.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/pylint/2.17.7/3.2.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/pylint/2.17.7/3.2.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>pylint-dev/pylint (pylint)</summary> ### [`v3.2.2`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.2.2) [Compare Source](https://togithub.com/pylint-dev/pylint/compare/v3.2.1...v3.2.2) ## What's new in Pylint 3.2.2? Release date: 2024-05-20 ## False Positives Fixed - Fix multiple false positives for generic class syntax added in Python 3.12 (PEP 695). Closes [#​9406](https://togithub.com/pylint-dev/pylint/issues/9406) - Exclude context manager without cleanup from `contextmanager-generator-missing-cleanup` checks. Closes [#​9625](https://togithub.com/pylint-dev/pylint/issues/9625) ### [`v3.2.1`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.2.1) [Compare Source](https://togithub.com/pylint-dev/pylint/compare/v3.2.0...v3.2.1) ## What's new in Pylint 3.2.1? Release date: 2024-05-18 ## False Positives Fixed - Exclude if/else branches containing terminating functions (e.g. `sys.exit()`) from `possibly-used-before-assignment` checks. Closes [#​9627](https://togithub.com/pylint-dev/pylint/issues/9627) - Don't emit `typevar-name-incorrect-variance` warnings for PEP 695 style TypeVars. The variance is inferred automatically by the type checker. Adding `_co` or `_contra` suffix can help to reason about TypeVar. Refs [#​9638](https://togithub.com/pylint-dev/pylint/issues/9638) - Fix a false positive for `possibly-used-before-assignment` when using `typing.assert_never()` (3.11+) to indicate exhaustiveness. Closes [#​9643](https://togithub.com/pylint-dev/pylint/issues/9643) ## Other Bug Fixes - Fix a false negative for `--ignore-patterns` when the directory to be linted is specified using a dot(`.`) and all files are ignored instead of only the files whose name begin with a dot. Closes [#​9273](https://togithub.com/pylint-dev/pylint/issues/9273) - Restore "errors / warnings by module" section to report output (with `-ry`). Closes [#​9145](https://togithub.com/pylint-dev/pylint/issues/9145) - `trailing-comma-tuple` should now be correctly emitted when it was disabled globally but enabled via local message control, after removal of an over-optimisation. Refs [#​9608](https://togithub.com/pylint-dev/pylint/issues/9608) - Add `--prefer-stubs=yes` option to opt-in to the astroid 3.2 feature that prefers `.pyi` stubs over same-named `.py` files. This has the potential to reduce `no-member` errors but at the cost of more errors such as `not-an-iterable` from function bodies appearing as `...`. Defaults to `no`. Closes [#​9626](https://togithub.com/pylint-dev/pylint/issues/9626) Closes [#​9623](https://togithub.com/pylint-dev/pylint/issues/9623) ## Internal Changes - Update astroid version to 3.2.1. This solves some reports of `RecursionError` and also makes the *prefer .pyi stubs* feature in astroid 3.2.0 *opt-in* with the aforementioned `--prefer-stubs=y` option. Refs [#​9139](https://togithub.com/pylint-dev/pylint/issues/9139) ### [`v3.2.0`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.2.0) [Compare Source](https://togithub.com/pylint-dev/pylint/compare/v3.1.1...v3.2.0) ## What's new in Pylint 3.2.0? Release date: 2024-05-14 Of note: a `github` reporter, two new checks (`possibly-used-before-assignment` and `contextmanager-generator-missing-cleanup`), performance improvements, and an `astroid` upgrade providing support for `@overload` and `.pyi` stubs. ## New Features - Understand `six.PY2` and `six.PY3` for conditional imports. Closes [#​3501](https://togithub.com/pylint-dev/pylint/issues/3501) - A new `github` reporter has been added. This reporter returns the output of `pylint` in a format that Github can use to automatically annotate code. Use it with `pylint --output-format=github` on your Github Workflows. Closes [#​9443](https://togithub.com/pylint-dev/pylint/issues/9443). ## New Checks - Add check `possibly-used-before-assignment` when relying on names after an `if/else` switch when one branch failed to define the name, raise, or return. Closes [#​1727](https://togithub.com/pylint-dev/pylint/issues/1727) - Checks for generators that use contextmanagers that don't handle cleanup properly. Is meant to raise visibilty on the case that a generator is not fully exhausted and the contextmanager is not cleaned up properly. A contextmanager must yield a non-constant value and not handle cleanup for GeneratorExit. The using generator must attempt to use the yielded context value `with x() as y` and not just `with x()`. Closes [#​2832](https://togithub.com/pylint-dev/pylint/issues/2832) ## False Negatives Fixed - If and Try nodes are now checked for useless return statements as well. Closes [#​9449](https://togithub.com/pylint-dev/pylint/issues/9449). - Fix false negative for `property-with-parameters` in the case of parameters which are `positional-only`, `keyword-only`, `variadic positional` or `variadic keyword`. Closes [#​9584](https://togithub.com/pylint-dev/pylint/issues/9584) ## False Positives Fixed - pylint now understands the `@overload` decorator return values better. Closes [#​4696](https://togithub.com/pylint-dev/pylint/issues/4696) Refs [#​9606](https://togithub.com/pylint-dev/pylint/issues/9606) ## Performance Improvements - Ignored modules are now not checked at all, instead of being checked and then ignored. This should speed up the analysis of large codebases which have ignored modules. Closes [#​9442](https://togithub.com/pylint-dev/pylint/issues/9442) - ImportChecker's logic has been modified to avoid context files when possible. This makes it possible to cache module searches on astroid and reduce execution times. Refs [#​9310](https://togithub.com/pylint-dev/pylint/issues/9310). - An internal check for `trailing-comma-tuple` being enabled for a file or not is now done once per file instead of once for each token. Refs [#​9608](https://togithub.com/pylint-dev/pylint/issues/9608). ### [`v3.1.1`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.1.1) [Compare Source](https://togithub.com/pylint-dev/pylint/compare/v3.1.0...v3.1.1) ## What's new in Pylint 3.1.1? Release date: 2024-05-13 ## False Positives Fixed - Treat `attrs.define` and `attrs.frozen` as dataclass decorators in `too-few-public-methods` check. Closes [#​9345](https://togithub.com/pylint-dev/pylint/issues/9345) - Fix a false positive with `singledispatchmethod-function` when a method is decorated with both `functools.singledispatchmethod` and `staticmethod`. Closes [#​9531](https://togithub.com/pylint-dev/pylint/issues/9531) - Fix a false positive for `consider-using-dict-items` when iterating using `keys()` and then deleting an item using the key as a lookup. Closes [#​9554](https://togithub.com/pylint-dev/pylint/issues/9554) ### [`v3.1.0`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.1.0) [Compare Source](https://togithub.com/pylint-dev/pylint/compare/v3.0.4...v3.1.0) Two new checks--`use-yield-from`, `deprecated-attribute`-- and a smattering of bug fixes. ## New Features - Skip `consider-using-join` check for non-empty separators if an `suggest-join-with-non-empty-separator` option is set to `no`. Closes [#​8701](https://togithub.com/pylint-dev/pylint/issues/8701) - Discover `.pyi` files when linting. These can be ignored with the `ignore-patterns` setting. Closes [#​9097](https://togithub.com/pylint-dev/pylint/issues/9097) - Check `TypeAlias` and `TypeVar` (PEP 695) nodes for `invalid-name`. Refs [#​9196](https://togithub.com/pylint-dev/pylint/issues/9196) - Support for resolving external toml files named pylintrc.toml and .pylintrc.toml. Closes [#​9228](https://togithub.com/pylint-dev/pylint/issues/9228) - Check for `.clear`, `.discard`, `.pop` and `remove` methods being called on a set while it is being iterated over. Closes [#​9334](https://togithub.com/pylint-dev/pylint/issues/9334) ## New Checks - New message `use-yield-from` added to the refactoring checker. This message is emitted when yielding from a loop can be replaced by `yield from`. Closes [#​9229](https://togithub.com/pylint-dev/pylint/issues/9229). - Added a `deprecated-attribute` message to check deprecated attributes in the stdlib. Closes [#​8855](https://togithub.com/pylint-dev/pylint/issues/8855) ## False Positives Fixed - Fixed false positive for `inherit-non-class` for generic Protocols. Closes [#​9106](https://togithub.com/pylint-dev/pylint/issues/9106) - Exempt `TypedDict` from `typing_extensions` from `too-many-ancestor` checks. Refs [#​9167](https://togithub.com/pylint-dev/pylint/issues/9167) ## False Negatives Fixed - Extend broad-exception-raised and broad-exception-caught to except\*. Closes [#​8827](https://togithub.com/pylint-dev/pylint/issues/8827) - Fix a false-negative for unnecessary if blocks using a different than expected ordering of arguments. Closes [#​8947](https://togithub.com/pylint-dev/pylint/issues/8947). ## Other Bug Fixes - Improve the message provided for wrong-import-order check. Instead of the import statement ("import x"), the message now specifies the import that is out of order and which imports should come after it. As reported in the issue, this is particularly helpful if there are multiple imports on a single line that do not follow the PEP8 convention. The message will report imports as follows: For "import X", it will report "(standard/third party/first party/local) import X" For "import X.Y" and "from X import Y", it will report "(standard/third party/first party/local) import X.Y" The import category is specified to provide explanation as to why pylint has issued the message and guidence to the developer on how to fix the problem. Closes [#​8808](https://togithub.com/pylint-dev/pylint/issues/8808) ## Other Changes - Print how many files were checked in verbose mode. Closes [#​8935](https://togithub.com/pylint-dev/pylint/issues/8935) - Fix a crash when an enum class which is also decorated with a `dataclasses.dataclass` decorator is defined. Closes [#​9100](https://togithub.com/pylint-dev/pylint/issues/9100) ## Internal Changes - Update astroid version to 3.1.0. Refs [#​9457](https://togithub.com/pylint-dev/pylint/issues/9457) ### [`v3.0.4`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.0.4) [Compare Source](https://togithub.com/pylint-dev/pylint/compare/v3.0.3...v3.0.4) ## False Positives Fixed - `used-before-assignment` is no longer emitted when using a name in a loop and depending on an earlier name assignment in an `except` block paired with `else: continue`. Closes [#​6804](https://togithub.com/pylint-dev/pylint/issues/6804) - Avoid false positives for `no-member` involving function attributes supplied by decorators. Closes [#​9246](https://togithub.com/pylint-dev/pylint/issues/9246) - Fixed false positive nested-min-max for nested lists. Closes [#​9307](https://togithub.com/pylint-dev/pylint/issues/9307) - Fix false positive for `used-before-assignment` in a `finally` block when assignments took place in both the `try` block and each exception handler. Closes [#​9451](https://togithub.com/pylint-dev/pylint/issues/9451) ## Other Bug Fixes - Catch incorrect ValueError `"generator already executing"` for Python 3.12.0 - 3.12.2. This is fixed upstream in Python 3.12.3. Closes [#​9138](https://togithub.com/pylint-dev/pylint/issues/9138) ### [`v3.0.3`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.0.3) [Compare Source](https://togithub.com/pylint-dev/pylint/compare/v3.0.2...v3.0.3) ## What's new in Pylint 3.0.3? Release date: 2023-12-11 ## False Positives Fixed - Fixed false positive for `unnecessary-lambda` when the call has keyword arguments but not the lambda. Closes [#​9148](https://togithub.com/pylint-dev/pylint/issues/9148) - Fixed incorrect suggestion for shallow copy in unnecessary-comprehension Example of the suggestion: \#pylint: disable=missing-module-docstring a = \[1, 2, 3] b = \[x for x in a] b\[0] = 0 print(a) # \[1, 2, 3] After changing b = \[x for x in a] to b = a based on the suggestion, the script now prints \[0, 2, 3]. The correct suggestion should be use list(a) to preserve the original behavior. Closes [#​9172](https://togithub.com/pylint-dev/pylint/issues/9172) - Fix false positives for `undefined-variable` and `unused-argument` for classes and functions using Python 3.12 generic type syntax. Closes [#​9193](https://togithub.com/pylint-dev/pylint/issues/9193) - Fixed `pointless-string-statement` false positive for docstrings on Python 3.12 type aliases. Closes [#​9268](https://togithub.com/pylint-dev/pylint/issues/9268) - Fix false positive for `invalid-exception-operation` when concatenating tuples of exception types. Closes [#​9288](https://togithub.com/pylint-dev/pylint/issues/9288) ## Other Bug Fixes - Fix a bug where pylint was unable to walk recursively through a directory if the directory has an `__init__.py` file. Closes [#​9210](https://togithub.com/pylint-dev/pylint/issues/9210) ### [`v3.0.2`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.0.2) [Compare Source](https://togithub.com/pylint-dev/pylint/compare/v3.0.1...v3.0.2) ## False Positives Fixed - Fix `used-before-assignment` false positive for generic type syntax (PEP 695, Python 3.12). Closes [#​9110](https://togithub.com/pylint-dev/pylint/issues/9110) ## Other Bug Fixes - Escape special symbols and newlines in messages. Closes [#​7874](https://togithub.com/pylint-dev/pylint/issues/7874) - Fixes suggestion for `nested-min-max` for expressions with additive operators, list and dict comprehensions. Closes [#​8524](https://togithub.com/pylint-dev/pylint/issues/8524) - Fixes ignoring conditional imports with `ignore-imports=y`. Closes [#​8914](https://togithub.com/pylint-dev/pylint/issues/8914) - Emit `inconsistent-quotes` for f-strings with 3.12 interpreter only if targeting pre-3.12 versions. Closes [#​9113](https://togithub.com/pylint-dev/pylint/issues/9113) ### [`v3.0.1`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.0.1) ## False Positives Fixed - Fixed false positive for `inherit-non-class` for generic Protocols. Closes [#​9106](https://togithub.com/pylint-dev/pylint/issues/9106) ## Other Changes - Fix a crash when an enum class which is also decorated with a `dataclasses.dataclass` decorator is defined. Closes [#​9100](https://togithub.com/pylint-dev/pylint/issues/9100) ### [`v3.0.0`](https://togithub.com/pylint-dev/pylint/releases/tag/v3.0.0) Pylint now support python 3.12 officially. This long anticipated major version also provides some important usability and performance improvements, along with enacting necessary breaking changes and long-announced deprecations. The documentation of each message with an example is very close too. The required astroid version is now 3.0.0. See the [astroid changelog](https://pylint.readthedocs.io/projects/astroid/en/latest/changelog.html#what-s-new-in-astroid-3-0-0) for additional fixes, features, and performance improvements applicable to pylint. Our code is now fully typed. The invalid-name message no longer checks for a minimum length of 3 characters by default. Dependencies like wrapt or setuptools were removed. A new json2 reporter has been added. It features an enriched output that is easier to parse and provides more info, here's a sample output. ```json { "messages": [ { "type": "convention", "symbol": "line-too-long", "message": "Line too long (1/2)", "messageId": "C0301", "confidence": "HIGH", "module": "0123", "obj": "", "line": 1, "column": 0, "endLine": 1, "endColumn": 4, "path": "0123", "absolutePath": "0123" } ], "statistics": { "messageTypeCount": { "fatal": 0, "error": 0, "warning": 0, "refactor": 0, "convention": 1, "info": 0 }, "modulesLinted": 1, "score": 5.0 } } ``` ## Breaking Changes - Enabling or disabling individual messages will now take effect even if an `--enable=all` or `disable=all` follows in the same configuration file (or on the command line). This means for the following example, `fixme` messages will now be emitted: ```bash pylint my_module --enable=fixme --disable=all ``` To regain the prior behavior, remove the superfluous earlier option. Closes [#​3696](https://togithub.com/pylint-dev/pylint/issues/3696) - Remove support for launching pylint with Python 3.7. Code that supports Python 3.7 can still be linted with the `--py-version=3.7` setting. Refs [#​6306](https://togithub.com/pylint-dev/pylint/issues/6306) - Disables placed in a `try` block now apply to the `except` block. Previously, they only happened to do so in the presence of an `else` clause. Refs [#​7767](https://togithub.com/pylint-dev/pylint/issues/7767) - `pyreverse` now uses a new default color palette that is more colorblind friendly. The color scheme is taken from `Paul Tol's Notes <https://personal.sron.nl/~pault/>`\_. If you prefer other colors, you can use the `--color-palette` option to specify custom colors. Closes [#​8251](https://togithub.com/pylint-dev/pylint/issues/8251) - Everything related to the `__implements__` construct was removed. It was based on PEP245 that was proposed in 2001 and rejected in 2006. The capability from pyreverse to take `__implements__` into account when generating diagrams was also removed. Refs [#​8404](https://togithub.com/pylint-dev/pylint/issues/8404) - `pyreverse`: Support for the `.vcg` output format (Visualization of Compiler Graphs) has been dropped. Closes [#​8416](https://togithub.com/pylint-dev/pylint/issues/8416) - The warning when the now useless old pylint cache directory (pylint.d) was found was removed. The cache dir is documented in `the FAQ <https://pylint.readthedocs.io/en/latest/faq.html#where-is-the-persistent-data-stored-to-compare-between-successive-runs>`\_. Refs [#​8462](https://togithub.com/pylint-dev/pylint/issues/8462) - Following a deprecation period, `pylint.config.PYLINTRC` was removed. Use the `pylint.config.find_default_config_files` generator instead. Closes [#​8862](https://togithub.com/pylint-dev/pylint/issues/8862) ## Changes requiring user actions - The `invalid-name` message no longer checks for a minimum length of 3 characters by default. (This was an unadvertised commingling of concerns between casing and name length, and users regularly reported this to be surprising.) If checking for a minimum length is still desired, it can be regained in two ways: - If you are content with a `disallowed-name` message (instead of `invalid-name`), then simply add the option `bad-names-rgxs="^..?$"`, which will fail 1-2 character-long names. (Ensure you enable `disallowed-name`.) - If you would prefer an `invalid-name` message to be emitted, or would prefer finer-grained control over the circumstances in which messages are emitted (classes vs. methods, etc.), then avail yourself of the regex options described `here <https://pylint.readthedocs.io/en/stable/user_guide/configuration/all-options.html#main-checker>`*. (In particular, take note of the commented out options in the "example configuration" given at the bottom of the section.) The prior regexes can be found in the `pull request <https://github.com/pylint-dev/pylint/pull/8813>`* that removed the length requirements. Closes [#​2018](https://togithub.com/pylint-dev/pylint/issues/2018) - The compare to empty string checker (`pylint.extensions.emptystring`) and the compare to zero checker (`pylint.extensions.compare-to-zero`) have been removed and their checks are now part of the implicit booleaness checker: - `compare-to-zero` was renamed `use-implicit-booleaness-not-comparison-to-zero` and `compare-to-empty-string` was renamed `use-implicit-booleaness-not-comparison-to-string` and they now need to be enabled explicitly. - The `pylint.extensions.emptystring` and `pylint.extensions.compare-to-zero` extensions no longer exist and need to be removed from the `load-plugins` option. - Messages related to implicit booleaness were made more explicit and actionable. This permits to make their likeness explicit and will provide better performance as they share most of their conditions to be raised. Closes [#​6871](https://togithub.com/pylint-dev/pylint/issues/6871) - epylint was removed. It now lives at: https://github.com/emacsorphanage/pylint. Refs [#​7737](https://togithub.com/pylint-dev/pylint/issues/7737) - The `overgeneral-exceptions` option now only takes fully qualified names into account (`builtins.Exception` not `Exception`). If you overrode this option, you need to use the fully qualified name now. There's still a warning, but it will be removed in 3.1.0. Refs [#​8411](https://togithub.com/pylint-dev/pylint/issues/8411) - Following a deprecation period, it's no longer possible to use `MASTER` or `master` as configuration section in `setup.cfg` or `tox.ini`. It's bad practice to not start a section title with the tool name. Please use `pylint.main` instead. Refs [#​8465](https://togithub.com/pylint-dev/pylint/issues/8465) - Package stats are now printed when running Pyreverse and a `--verbose` flag was added to get the original output with parsed modules. You might need to activate the verbose option if you want to keep the old output. Closes [#​8973](https://togithub.com/pylint-dev/pylint/issues/8973) ## New Features - A new `json2` reporter has been added. It features a more enriched output that is easier to parse and provides more info. Compared to `json` the only changes are that messages are now under the `"messages"` key and that `"message-id"` now follows the camelCase convention and is renamed to `"messageId"`. The new reporter also reports the "score" of the modules you linted as defined by the `evaluation` option and provides statistics about the modules you linted. We encourage users to use the new reporter as the `json` reporter will no longer be maintained. Closes [#​4741](https://togithub.com/pylint-dev/pylint/issues/4741) - In Pyreverse package dependency diagrams, show when a module imports another only for type-checking. Closes [#​8112](https://togithub.com/pylint-dev/pylint/issues/8112) - Add new option (`--show-stdlib`, `-L`) to `pyreverse`. This is similar to the behavior of `--show-builtin` in that standard library modules are now not included by default, and this option will include them. Closes [#​8181](https://togithub.com/pylint-dev/pylint/issues/8181) - Add Pyreverse option to exclude standalone nodes from diagrams with `--no-standalone`. Closes [#​8476](https://togithub.com/pylint-dev/pylint/issues/8476) ## New Checks - Added `DataclassChecker` module and `invalid-field-call` checker to check for invalid dataclasses.field() usage. Refs [#​5159](https://togithub.com/pylint-dev/pylint/issues/5159) - Add `return-in-finally` to emit a message if a return statement was found in a finally clause. Closes [#​8260](https://togithub.com/pylint-dev/pylint/issues/8260) - Add a new checker `kwarg-superseded-by-positional-arg` to warn when a function is called with a keyword argument which shares a name with a positional-only parameter and the function contains a keyword variadic parameter dictionary. It may be surprising behaviour when the keyword argument is added to the keyword variadic parameter dictionary. Closes [#​8558](https://togithub.com/pylint-dev/pylint/issues/8558) ## Extensions - Add new `prefer-typing-namedtuple` message to the `CodeStyleChecker` to suggest rewriting calls to `collections.namedtuple` as classes inheriting from `typing.NamedTuple` on Python 3.6+. Requires `load-plugins=pylint.extensions.code_style` and `enable=prefer-typing-namedtuple` to be raised. Closes [#​8660](https://togithub.com/pylint-dev/pylint/issues/8660) ## False Positives Fixed - Extend concept of "function ambiguity" in `safe_infer()` from differing number of function arguments to differing set of argument names. Solves false positives in `tensorflow`. Closes [#​3613](https://togithub.com/pylint-dev/pylint/issues/3613) - Fix `unused-argument` false positive when `__new__` does not use all the arguments of `__init__`. Closes [#​3670](https://togithub.com/pylint-dev/pylint/issues/3670) - Fix a false positive for `invalid-name` when a type-annotated class variable in an `enum.Enum` class has no assigned value. Refs [#​7402](https://togithub.com/pylint-dev/pylint/issues/7402) - Fix `unused-import` false positive for usage of `six.with_metaclass`. Closes [#​7506](https://togithub.com/pylint-dev/pylint/issues/7506) - Fix false negatives and false positives for `too-many-try-statements`, `too-complex`, and `too-many-branches` by correctly counting statements under a `try`. Refs [#​7767](https://togithub.com/pylint-dev/pylint/issues/7767) - When checking for unbalanced dict unpacking in for-loops, Pylint will now test whether the length of each value to be unpacked matches the number of unpacking targets. Previously, Pylint would test the number of values for the loop iteration, which would produce a false unbalanced-dict-unpacking warning. Closes [#​8156](https://togithub.com/pylint-dev/pylint/issues/8156) - Fix false positive for `used-before-assignment` when usage and assignment are guarded by the same test in different statements. Closes [#​8167](https://togithub.com/pylint-dev/pylint/issues/8167) - Adds `asyncSetUp` to the default `defining-attr-methods` list to silence `attribute-defined-outside-init` warning when using `unittest.IsolatedAsyncioTestCase`. Refs [#​8403](https://togithub.com/pylint-dev/pylint/issues/8403) - `logging-not-lazy` is not longer emitted for explicitly concatenated string arguments. Closes [#​8410](https://togithub.com/pylint-dev/pylint/issues/8410) - Fix false positive for isinstance-second-argument-not-valid-type when union types contains None. Closes [#​8424](https://togithub.com/pylint-dev/pylint/issues/8424) - `invalid-name` now allows for integers in `typealias` names: - now valid: `Good2Name`, `GoodName2`. - still invalid: `_1BadName`. Closes [#​8485](https://togithub.com/pylint-dev/pylint/issues/8485) - No longer consider `Union` as type annotation as type alias for naming checks. Closes [#​8487](https://togithub.com/pylint-dev/pylint/issues/8487) - `unnecessary-lambda` no longer warns on lambdas which use its parameters in their body (other than the final arguments), e.g. `lambda foo: (bar if foo else baz)(foo)`. Closes [#​8496](https://togithub.com/pylint-dev/pylint/issues/8496) - Fixed `unused-import` so that it observes the `dummy-variables-rgx` option. Closes [#​8500](https://togithub.com/pylint-dev/pylint/issues/8500) - `Union` typed variables without assignment are no longer treated as `TypeAlias`. Closes [#​8540](https://togithub.com/pylint-dev/pylint/issues/8540) - Allow parenthesized implicitly concatenated strings when `check-str-concat-over-line-jumps` is enabled. Closes [#​8552](https://togithub.com/pylint-dev/pylint/issues/8552). - Fix false positive for `positional-only-arguments-expected` when a function contains both a positional-only parameter that has a default value, and `**kwargs`. Closes [#​8555](https://togithub.com/pylint-dev/pylint/issues/8555) - Fix false positive for `keyword-arg-before-vararg` when a positional-only parameter with a default value precedes `*args`. Closes [#​8570](https://togithub.com/pylint-dev/pylint/issues/8570) - Fix false positive for `arguments-differ` when overriding `__init_subclass__`. Closes [#​8919](https://togithub.com/pylint-dev/pylint/issues/8919) - Fix a false positive for `no-value-for-parameter` when a staticmethod is called in a class body. Closes [#​9036](https://togithub.com/pylint-dev/pylint/issues/9036) ## False Negatives Fixed - Emit `used-before-assignment` when calling module-level functions before definition. Closes [#​1144](https://togithub.com/pylint-dev/pylint/issues/1144) - Apply `infer_kwarg_from_call()` to more checks These mostly solve false negatives for various checks, save for one false positive for `use-maxsplit-arg`. Closes [#​7761](https://togithub.com/pylint-dev/pylint/issues/7761) - `TypeAlias` variables defined in functions are now checked for `invalid-name` errors. Closes [#​8536](https://togithub.com/pylint-dev/pylint/issues/8536) - Fix false negative for `no-value-for-parameter` when a function, whose signature contains both a positional-only parameter `name` and also `*kwargs`, is called with a keyword-argument for `name`. Closes [#​8559](https://togithub.com/pylint-dev/pylint/issues/8559) - Fix a false negative for `too-many-arguments` by considering positional-only and keyword-only parameters. Closes [#​8667](https://togithub.com/pylint-dev/pylint/issues/8667) - Emit `assignment-from-no-return` for calls to builtin methods like `dict.update()`. Calls to `list.sort()` now raise `assignment-from-no-return` rather than `assignment-from-none` for consistency. Closes [#​8714](https://togithub.com/pylint-dev/pylint/issues/8714) Closes [#​8810](https://togithub.com/pylint-dev/pylint/issues/8810) - `consider-using-augmented-assign` is now applied to dicts and lists as well. Closes [#​8959](https://togithub.com/pylint-dev/pylint/issues/8959) ## Other Bug Fixes - Support `duplicate-code` message when parallelizing with `--jobs`. Closes [#​374](https://togithub.com/pylint-dev/pylint/issues/374) - Support `cyclic-import` message when parallelizing with `--jobs`. Closes [#​4171](https://togithub.com/pylint-dev/pylint/issues/4171) - `--jobs` can now be used with `--load-plugins`. This had regressed in astroid 2.5.0. Closes [#​4874](https://togithub.com/pylint-dev/pylint/issues/4874) - docparams extension considers type comments as type documentation. Closes [#​6287](https://togithub.com/pylint-dev/pylint/issues/6287) - When parsing comma-separated lists of regular expressions in the config, ignore commas that are inside braces since those indicate quantifiers, not delineation between expressions. Closes [#​7229](https://togithub.com/pylint-dev/pylint/issues/7229) - The `ignored-modules` option will now be correctly taken into account for `no-name-in-module`. Closes [#​7578](https://togithub.com/pylint-dev/pylint/issues/7578) - `sys.argv` is now always correctly considered as impossible to infer (instead of using the actual values given to pylint). Closes [#​7710](https://togithub.com/pylint-dev/pylint/issues/7710) - Avoid duplicative warnings for unqualified exception names in the `overgeneral-exceptions` setting when running with `--jobs`. Closes [#​7774](https://togithub.com/pylint-dev/pylint/issues/7774) - Don't show class fields more than once in Pyreverse diagrams. Closes [#​8189](https://togithub.com/pylint-dev/pylint/issues/8189) - Fix `used-before-assignment` false negative when TYPE_CHECKING imports are used in multiple scopes. Closes [#​8198](https://togithub.com/pylint-dev/pylint/issues/8198) - `--clear-cache-post-run` now also clears LRU caches for pylint utilities holding references to AST nodes. Closes [#​8361](https://togithub.com/pylint-dev/pylint/issues/8361) - Fix a crash when `TYPE_CHECKING` is used without importing it. Closes [#​8434](https://togithub.com/pylint-dev/pylint/issues/8434) - Fix a `used-before-assignment` false positive when imports are made under the `TYPE_CHECKING` else if branch. Closes [#​8437](https://togithub.com/pylint-dev/pylint/issues/8437) - Fix a regression of `preferred-modules` where a partial match was used instead of the required full match. Closes [#​8453](https://togithub.com/pylint-dev/pylint/issues/8453) - Fix a crash in pyreverse when "/" characters are used in the output filename e.g pyreverse -o png -p name/ path/to/project. Closes [#​8504](https://togithub.com/pylint-dev/pylint/issues/8504) - Don't show arrows more than once in Pyreverse diagrams. Closes [#​8522](https://togithub.com/pylint-dev/pylint/issues/8522) - Improve output of `consider-using-generator` message for `min()` calls with `default` keyword. Closes [#​8563](https://togithub.com/pylint-dev/pylint/issues/8563) - Fixed a crash when generating a configuration file: `tomlkit.exceptions.TOMLKitError: Can't add a table to a dotted key` caused by tomlkit `v0.11.8`. Closes [#​8632](https://togithub.com/pylint-dev/pylint/issues/8632) - Fix a line break error in Pyreverse dot output. Closes [#​8671](https://togithub.com/pylint-dev/pylint/issues/8671) - Fix a false positive for `method-hidden` when using `cached_property` decorator. Closes [#​8753](https://togithub.com/pylint-dev/pylint/issues/8753) - Dunder methods defined in lambda do not trigger `unnecessary-dunder-call` anymore, if they cannot be replaced by the non-dunder call. Closes [#​8769](https://togithub.com/pylint-dev/pylint/issues/8769) - Don't show duplicate type annotations in Pyreverse diagrams. Closes [#​8888](https://togithub.com/pylint-dev/pylint/issues/8888) - Fixing inconsistent hashing issue in `BaseChecker` causing some reports not being exported. Closes [#​9001](https://togithub.com/pylint-dev/pylint/issues/9001) - Don't add `Optional` to `|` annotations with `None` in Pyreverse diagrams. Closes [#​9014](https://togithub.com/pylint-dev/pylint/issues/9014) - Pyreverse doesn't show multiple class association arrows anymore, but only the strongest one. Refs [#​9045](https://togithub.com/pylint-dev/pylint/issues/9045) - Prevented data loss in the linter stats for messages relating to the linter itself (e.g. `unknown-option-value`), fixing problems with score, fail-on, etc. Closes [#​9059](https://togithub.com/pylint-dev/pylint/issues/9059) - Fix crash in refactoring checker when unary operand used with variable in for loop. Closes [#​9074](https://togithub.com/pylint-dev/pylint/issues/9074) ## Other Changes - Pylint now exposes its type annotations. Closes [#​5488](https://togithub.com/pylint-dev/pylint/issues/5488) and [#​2079](https://togithub.com/pylint-dev/pylint/issues/2079) - Search for `pyproject.toml` recursively in parent directories up to a project or file system root. Refs [#​7163](https://togithub.com/pylint-dev/pylint/issues/7163), Closes [#​3289](https://togithub.com/pylint-dev/pylint/issues/3289) - All code related to the optparse config parsing has been removed. Refs [#​8405](https://togithub.com/pylint-dev/pylint/issues/8405) - Pylint now supports python 3.12. Refs [#​8718](https://togithub.com/pylint-dev/pylint/issues/8718) - Add a CITATION.cff file to the root of the repository containing the necessary metadata to cite Pylint. Closes [#​8760](https://togithub.com/pylint-dev/pylint/issues/8760) - Renamed the "unneeded-not" error into "unnecessary_negation" to be clearer. Closes [#​8789](https://togithub.com/pylint-dev/pylint/issues/8789) ## Internal Changes - `get_message_definition` was removed from the base checker API. You can access message definitions through the `MessageStore`. Refs [#​8401](https://togithub.com/pylint-dev/pylint/issues/8401) - Everything related to the `__implements__` construct was removed. It was based on PEP245 that was proposed in 2001 and rejected in 2006. All the classes inheriting `Interface` in `pylint.interfaces` were removed. `Checker` should only inherit `BaseChecker` or any of the other checker types from `pylint.checkers`. `Reporter` should only inherit `BaseReporter`. Refs [#​8404](https://togithub.com/pylint-dev/pylint/issues/8404) - `modname` and `msg_store` are now required to be given in `FileState`. `collect_block_lines` has also been removed. `Pylinter.current_name` cannot be null anymore. Refs [#​8407](https://togithub.com/pylint-dev/pylint/issues/8407) - `Reporter.set_output` was removed in favor of `reporter.out = stream`. Refs [#​8408](https://togithub.com/pylint-dev/pylint/issues/8408) - A number of old utility functions and classes have been removed: `MapReduceMixin`: To make a checker reduce map data simply implement `get_map_data` and `reduce_map_data`. `is_inside_lambda`: Use `utils.get_node_first_ancestor_of_type(x, nodes.Lambda)` `check_messages`: Use `utils.only_required_for_messages` `is_class_subscriptable_pep585_with_postponed_evaluation_enabled`: Use `is_postponed_evaluation_enabled(node)` and `is_node_in_type_annotation_context(node)` `get_python_path`: assumption that there's always an **init**.py is not true since python 3.3 and is causing problems, particularly with PEP 420. Use `discover_package_path` and pass source root(s). `fix_import_path`: Use `augmented_sys_path` and pass additional `sys.path` entries as an argument obtained from `discover_package_path`. `get_global_option`: Use `checker.linter.config` to get all global options. Related private objects have been removed as well. Refs [#​8409](https://togithub.com/pylint-dev/pylint/issues/8409) - `colorize_ansi` now only accepts a `MessageStyle` object. Refs [#​8412](https://togithub.com/pylint-dev/pylint/issues/8412) - Following a deprecation period, `Pylinter.check` now only works with sequences of strings, not strings. Refs [#​8463](https://togithub.com/pylint-dev/pylint/issues/8463) - Following a deprecation period, `ColorizedTextReporter` only accepts `ColorMappingDict`. Refs [#​8464](https://togithub.com/pylint-dev/pylint/issues/8464) - Following a deprecation period, `MessageTest`'s `end_line` and `end_col_offset` must be accurate in functional tests (for python 3.8 or above on cpython, and for python 3.9 or superior on pypy). Refs [#​8466](https://togithub.com/pylint-dev/pylint/issues/8466) - Following a deprecation period, the `do_exit` argument of the `Run` class (and of the `_Run` class in testutils) were removed. Refs [#​8472](https://togithub.com/pylint-dev/pylint/issues/8472) - Following a deprecation period, the `py_version` argument of the `MessageDefinition.may_be_emitted` function is now required. The most likely solution is to use 'linter.config.py_version' if you need to keep using this function, or to use 'MessageDefinition.is_message_enabled' instead. Refs [#​8473](https://togithub.com/pylint-dev/pylint/issues/8473) - Following a deprecation period, the `OutputLine` class now requires the right number of argument all the time. The functional output can be regenerated automatically to achieve that easily. Refs [#​8474](https://togithub.com/pylint-dev/pylint/issues/8474) - Following a deprecation period, `is_typing_guard`, `is_node_in_typing_guarded_import_block` and `is_node_in_guarded_import_block` from `pylint.utils` were removed: use a combination of `is_sys_guard` and `in_type_checking_block` instead. Refs [#​8475](https://togithub.com/pylint-dev/pylint/issues/8475) - Following a deprecation period, the `location` argument of the `Message` class must now be a `MessageLocationTuple`. Refs [#​8477](https://togithub.com/pylint-dev/pylint/issues/8477) - Following a deprecation period, the `check_single_file` function of the `Pylinter` is replaced by `Pylinter.check_single_file_item`. Refs [#​8478](https://togithub.com/pylint-dev/pylint/issues/8478) ## Performance Improvements - `pylint` runs (at least) ~5% faster after improvements to `astroid` that make better use of the inference cache. Refs [pylint-dev/astroid#529](https://togithub.com/pylint-dev/astroid/issues/529) - - Optimize `is_trailing_comma()`. - Cache `class_is_abstract()`. Refs [#​1954](https://togithub.com/pylint-dev/pylint/issues/1954) - Exit immediately if all messages are disabled. Closes [#​8715](https://togithub.com/pylint-dev/pylint/issues/8715) </details> --- ### Configuration π **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). π¦ **Automerge**: Disabled by config. Please merge this manually once you are satisfied. β» **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. π **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/premiscale/premiscale). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xMDMuMSIsInVwZGF0ZWRJblZlciI6IjM3LjM3Ny44IiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIn0=-->
This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [pylint](https://redirect.github.com/pylint-dev/pylint) ([changelog](https://pylint.readthedocs.io/en/latest/whatsnew/3/)) | `==2.15.5` -> `==3.3.1` | [![age](https://developer.mend.io/api/mc/badges/age/pypi/pylint/3.3.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/pylint/3.3.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/pylint/2.15.5/3.3.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/pylint/2.15.5/3.3.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>pylint-dev/pylint (pylint)</summary> ### [`v3.3.1`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.3.1) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.3.0...v3.3.1) ## What's new in Pylint 3.3.1? Release date: 2024-09-24 ## False Positives Fixed - Fix regression causing some f-strings to not be inferred as strings. Closes [#​9947](https://redirect.github.com/pylint-dev/pylint/issues/9947) ### [`v3.3.0`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.3.0) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.2.7...v3.3.0) Release date: 2024-09-20 ## Changes requiring user actions - We migrated `symilar` to argparse, from getopt, so the error and help output changed (for the better). We exit with 2 instead of sometime 1, sometime 2. The error output is not captured by the runner anymore. It's not possible to use a value for the boolean options anymore (`--ignore-comments 1` should become `--ignore-comments`). Refs [#​9731](https://redirect.github.com/pylint-dev/pylint/issues/9731) ## New Features - Add new `declare-non-slot` error which reports when a class has a `__slots__` member and a type hint on the class is not present in `__slots__`. Refs [#​9499](https://redirect.github.com/pylint-dev/pylint/issues/9499) ## New Checks - Added `too-many-positional-arguments` to allow distinguishing the configuration for too many total arguments (with keyword-only params specified after `*`) from the configuration for too many positional-or-keyword or positional-only arguments. As part of evaluating whether this check makes sense for your project, ensure you adjust the value of `--max-positional-arguments`. Closes [#​9099](https://redirect.github.com/pylint-dev/pylint/issues/9099) - Add `using-exception-group-in-unsupported-version` and `using-generic-type-syntax-in-unsupported-version` for uses of Python 3.11+ or 3.12+ features on lower supported versions provided with `--py-version`. Closes [#​9791](https://redirect.github.com/pylint-dev/pylint/issues/9791) - Add `using-assignment-expression-in-unsupported-version` for uses of `:=` (walrus operator) on Python versions below 3.8 provided with `--py-version`. Closes [#​9820](https://redirect.github.com/pylint-dev/pylint/issues/9820) - Add `using-positional-only-args-in-unsupported-version` for uses of positional-only args on Python versions below 3.8 provided with `--py-version`. Closes [#​9823](https://redirect.github.com/pylint-dev/pylint/issues/9823) - Add `unnecessary-default-type-args` to the `typing` extension to detect the use of unnecessary default type args for `typing.Generator` and `typing.AsyncGenerator`. Refs [#​9938](https://redirect.github.com/pylint-dev/pylint/issues/9938) ## False Negatives Fixed - Fix computation of never-returning function: `Never` is handled in addition to `NoReturn`, and priority is given to the explicit `--never-returning-functions` option. Closes [#​7565](https://redirect.github.com/pylint-dev/pylint/issues/7565). - Fix a false negative for `await-outside-async` when await is inside Lambda. Refs [#​9653](https://redirect.github.com/pylint-dev/pylint/issues/9653) - Fix a false negative for `duplicate-argument-name` by including `positional-only`, `*args` and `**kwargs` arguments in the check. Closes [#​9669](https://redirect.github.com/pylint-dev/pylint/issues/9669) - Fix false negative for `multiple-statements` when multiple statements are present on `else` and `finally` lines of `try`. Refs [#​9759](https://redirect.github.com/pylint-dev/pylint/issues/9759) - Fix false negatives when `isinstance` does not have exactly two arguments. pylint now emits a `too-many-function-args` or `no-value-for-parameter` appropriately for `isinstance` calls. Closes [#​9847](https://redirect.github.com/pylint-dev/pylint/issues/9847) ## Other Bug Fixes - `--enable` with `--disable=all` now produces an error, when an unknown msg code is used. Internal `pylint` messages are no longer affected by `--disable=all`. Closes [#​9403](https://redirect.github.com/pylint-dev/pylint/issues/9403) - Impossible to compile regexes for paths in the configuration or argument given to pylint won't crash anymore but raise an argparse error and display the error message from `re.compile` instead. Closes [#​9680](https://redirect.github.com/pylint-dev/pylint/issues/9680) - Fix a bug where a `tox.ini` file with pylint configuration was ignored and it exists in the current directory. `.cfg` and `.ini` files containing a `Pylint` configuration may now use a section named `[pylint]`. This enhancement impacts the scenario where these file types are used as defaults when they are present and have not been explicitly referred to, using the `--rcfile` option. Closes [#​9727](https://redirect.github.com/pylint-dev/pylint/issues/9727) - Improve file discovery for directories that are not python packages. Closes [#​9764](https://redirect.github.com/pylint-dev/pylint/issues/9764) ## Other Changes - Remove support for launching pylint with Python 3.8. Code that supports Python 3.8 can still be linted with the `--py-version=3.8` setting. Refs [#​9774](https://redirect.github.com/pylint-dev/pylint/issues/9774) - Add support for Python 3.13. Refs [#​9852](https://redirect.github.com/pylint-dev/pylint/issues/9852) ## Internal Changes - All variables, classes, functions and file names containing the word 'similar', when it was, in fact, referring to 'symilar' (the standalone program for the duplicate-code check) were renamed to 'symilar'. Closes [#​9734](https://redirect.github.com/pylint-dev/pylint/issues/9734) - Remove old-style classes (Python 2) code and remove check for new-style class since everything is new-style in Python 3. Updated doc for exception checker to remove reference to new style class. Refs [#​9925](https://redirect.github.com/pylint-dev/pylint/issues/9925) ### [`v3.2.7`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.2.7) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.2.6...v3.2.7) ## What's new in Pylint 3.2.7? Release date: 2024-08-31 ## False Positives Fixed - Fixed a false positive `unreachable` for `NoReturn` coroutine functions. Closes [#​9840](https://redirect.github.com/pylint-dev/pylint/issues/9840) ## Other Bug Fixes - Fix crash in refactoring checker when calling a lambda bound as a method. Closes [#​9865](https://redirect.github.com/pylint-dev/pylint/issues/9865) - Fix a crash in `undefined-loop-variable` when providing the `iterable` argument to `enumerate()`. Closes [#​9875](https://redirect.github.com/pylint-dev/pylint/issues/9875) - Fix to address indeterminacy of error message in case a module name is same as another in a separate namespace. Refs [#​9883](https://redirect.github.com/pylint-dev/pylint/issues/9883) ### [`v3.2.6`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.2.6) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.2.5...v3.2.6) ## What's new in Pylint 3.2.6? Release date: 2024-07-21 ## False Positives Fixed - Quiet false positives for `unexpected-keyword-arg` when pylint cannot determine which of two or more dynamically defined classes is being instantiated. Closes [#​9672](https://redirect.github.com/pylint-dev/pylint/issues/9672) - Fix a false positive for `missing-param-doc` where a method which is decorated with `typing.overload` was expected to have a docstring specifying its parameters. Closes [#​9739](https://redirect.github.com/pylint-dev/pylint/issues/9739) - Fix a regression that raised `invalid-name` on class attributes merely overriding invalid names from an ancestor. Closes [#​9765](https://redirect.github.com/pylint-dev/pylint/issues/9765) - Treat `assert_never()` the same way when imported from `typing_extensions`. Closes [#​9780](https://redirect.github.com/pylint-dev/pylint/issues/9780) - Fix a false positive for `consider-using-min-max-builtin` when the assignment target is an attribute. Refs [#​9800](https://redirect.github.com/pylint-dev/pylint/issues/9800) ## Other Bug Fixes - Fix an `AssertionError` arising from properties that return partial functions. Closes [#​9214](https://redirect.github.com/pylint-dev/pylint/issues/9214) - Fix a crash when a subclass extends `__slots__`. Closes [#​9814](https://redirect.github.com/pylint-dev/pylint/issues/9814) ### [`v3.2.5`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.2.5) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.2.4...v3.2.5) ## What's new in Pylint 3.2.5 ? Release date: 2024-06-28 ## Other Bug Fixes - Fixed a false positive `unreachable-code` when using `typing.Any` as return type in python 3.8, the `typing.NoReturn` are not taken into account anymore for python 3.8 however. Closes [#​9751](https://redirect.github.com/pylint-dev/pylint/issues/9751) ### [`v3.2.4`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.2.4) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.2.3...v3.2.4) ## What's new in Pylint 3.2.4? Release date: 2024-06-26 ## False Positives Fixed - Prevent emitting `possibly-used-before-assignment` when relying on names only potentially not defined in conditional blocks guarded by functions annotated with `typing.Never` or `typing.NoReturn`. Closes [#​9674](https://redirect.github.com/pylint-dev/pylint/issues/9674) ## Other Bug Fixes - Fixed a crash when the lineno of a variable used as an annotation wasn't available for `undefined-variable`. Closes [#​8866](https://redirect.github.com/pylint-dev/pylint/issues/8866) - Fixed a crash when the `start` value in an `enumerate` was non-constant and impossible to infer (like in`enumerate(apples, start=int(random_apple_index)`) for `unnecessary-list-index-lookup`. Closes [#​9078](https://redirect.github.com/pylint-dev/pylint/issues/9078) - Fixed a crash in `symilar` when the `-d` or `-i` short option were not properly recognized. It's still impossible to do `-d=1` (you must do `-d 1`). Closes [#​9343](https://redirect.github.com/pylint-dev/pylint/issues/9343) ### [`v3.2.3`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.2.3) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.2.2...v3.2.3) ## False Positives Fixed - Classes with only an Ellipsis (`...`) in their body do not trigger 'multiple-statements' anymore if they are inlined (in accordance with black's 2024 style). Closes [#​9398](https://redirect.github.com/pylint-dev/pylint/issues/9398) - Fix a false positive for `redefined-outer-name` when there is a name defined in an exception-handling block which shares the same name as a local variable that has been defined in a function body. Closes [#​9671](https://redirect.github.com/pylint-dev/pylint/issues/9671) - Fix a false positive for `use-yield-from` when using the return value from the `yield` atom. Closes [#​9696](https://redirect.github.com/pylint-dev/pylint/issues/9696) ### [`v3.2.2`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.2.2) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.2.1...v3.2.2) ## What's new in Pylint 3.2.2? Release date: 2024-05-20 ## False Positives Fixed - Fix multiple false positives for generic class syntax added in Python 3.12 (PEP 695). Closes [#​9406](https://redirect.github.com/pylint-dev/pylint/issues/9406) - Exclude context manager without cleanup from `contextmanager-generator-missing-cleanup` checks. Closes [#​9625](https://redirect.github.com/pylint-dev/pylint/issues/9625) ### [`v3.2.1`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.2.1) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.2.0...v3.2.1) ## What's new in Pylint 3.2.1? Release date: 2024-05-18 ## False Positives Fixed - Exclude if/else branches containing terminating functions (e.g. `sys.exit()`) from `possibly-used-before-assignment` checks. Closes [#​9627](https://redirect.github.com/pylint-dev/pylint/issues/9627) - Don't emit `typevar-name-incorrect-variance` warnings for PEP 695 style TypeVars. The variance is inferred automatically by the type checker. Adding `_co` or `_contra` suffix can help to reason about TypeVar. Refs [#​9638](https://redirect.github.com/pylint-dev/pylint/issues/9638) - Fix a false positive for `possibly-used-before-assignment` when using `typing.assert_never()` (3.11+) to indicate exhaustiveness. Closes [#​9643](https://redirect.github.com/pylint-dev/pylint/issues/9643) ## Other Bug Fixes - Fix a false negative for `--ignore-patterns` when the directory to be linted is specified using a dot(`.`) and all files are ignored instead of only the files whose name begin with a dot. Closes [#​9273](https://redirect.github.com/pylint-dev/pylint/issues/9273) - Restore "errors / warnings by module" section to report output (with `-ry`). Closes [#​9145](https://redirect.github.com/pylint-dev/pylint/issues/9145) - `trailing-comma-tuple` should now be correctly emitted when it was disabled globally but enabled via local message control, after removal of an over-optimisation. Refs [#​9608](https://redirect.github.com/pylint-dev/pylint/issues/9608) - Add `--prefer-stubs=yes` option to opt-in to the astroid 3.2 feature that prefers `.pyi` stubs over same-named `.py` files. This has the potential to reduce `no-member` errors but at the cost of more errors such as `not-an-iterable` from function bodies appearing as `...`. Defaults to `no`. Closes [#​9626](https://redirect.github.com/pylint-dev/pylint/issues/9626) Closes [#​9623](https://redirect.github.com/pylint-dev/pylint/issues/9623) ## Internal Changes - Update astroid version to 3.2.1. This solves some reports of `RecursionError` and also makes the *prefer .pyi stubs* feature in astroid 3.2.0 *opt-in* with the aforementioned `--prefer-stubs=y` option. Refs [#​9139](https://redirect.github.com/pylint-dev/pylint/issues/9139) ### [`v3.2.0`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.2.0) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.1.1...v3.2.0) ## What's new in Pylint 3.2.0? Release date: 2024-05-14 Of note: a `github` reporter, two new checks (`possibly-used-before-assignment` and `contextmanager-generator-missing-cleanup`), performance improvements, and an `astroid` upgrade providing support for `@overload` and `.pyi` stubs. ## New Features - Understand `six.PY2` and `six.PY3` for conditional imports. Closes [#​3501](https://redirect.github.com/pylint-dev/pylint/issues/3501) - A new `github` reporter has been added. This reporter returns the output of `pylint` in a format that Github can use to automatically annotate code. Use it with `pylint --output-format=github` on your Github Workflows. Closes [#​9443](https://redirect.github.com/pylint-dev/pylint/issues/9443). ## New Checks - Add check `possibly-used-before-assignment` when relying on names after an `if/else` switch when one branch failed to define the name, raise, or return. Closes [#​1727](https://redirect.github.com/pylint-dev/pylint/issues/1727) - Checks for generators that use contextmanagers that don't handle cleanup properly. Is meant to raise visibilty on the case that a generator is not fully exhausted and the contextmanager is not cleaned up properly. A contextmanager must yield a non-constant value and not handle cleanup for GeneratorExit. The using generator must attempt to use the yielded context value `with x() as y` and not just `with x()`. Closes [#​2832](https://redirect.github.com/pylint-dev/pylint/issues/2832) ## False Negatives Fixed - If and Try nodes are now checked for useless return statements as well. Closes [#​9449](https://redirect.github.com/pylint-dev/pylint/issues/9449). - Fix false negative for `property-with-parameters` in the case of parameters which are `positional-only`, `keyword-only`, `variadic positional` or `variadic keyword`. Closes [#​9584](https://redirect.github.com/pylint-dev/pylint/issues/9584) ## False Positives Fixed - pylint now understands the `@overload` decorator return values better. Closes [#​4696](https://redirect.github.com/pylint-dev/pylint/issues/4696) Refs [#​9606](https://redirect.github.com/pylint-dev/pylint/issues/9606) ## Performance Improvements - Ignored modules are now not checked at all, instead of being checked and then ignored. This should speed up the analysis of large codebases which have ignored modules. Closes [#​9442](https://redirect.github.com/pylint-dev/pylint/issues/9442) - ImportChecker's logic has been modified to avoid context files when possible. This makes it possible to cache module searches on astroid and reduce execution times. Refs [#​9310](https://redirect.github.com/pylint-dev/pylint/issues/9310). - An internal check for `trailing-comma-tuple` being enabled for a file or not is now done once per file instead of once for each token. Refs [#​9608](https://redirect.github.com/pylint-dev/pylint/issues/9608). ### [`v3.1.1`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.1.1) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.1.0...v3.1.1) ## What's new in Pylint 3.1.1? Release date: 2024-05-13 ## False Positives Fixed - Treat `attrs.define` and `attrs.frozen` as dataclass decorators in `too-few-public-methods` check. Closes [#​9345](https://redirect.github.com/pylint-dev/pylint/issues/9345) - Fix a false positive with `singledispatchmethod-function` when a method is decorated with both `functools.singledispatchmethod` and `staticmethod`. Closes [#​9531](https://redirect.github.com/pylint-dev/pylint/issues/9531) - Fix a false positive for `consider-using-dict-items` when iterating using `keys()` and then deleting an item using the key as a lookup. Closes [#​9554](https://redirect.github.com/pylint-dev/pylint/issues/9554) ### [`v3.1.0`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.1.0) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.0.4...v3.1.0) Two new checks--`use-yield-from`, `deprecated-attribute`-- and a smattering of bug fixes. ## New Features - Skip `consider-using-join` check for non-empty separators if an `suggest-join-with-non-empty-separator` option is set to `no`. Closes [#​8701](https://redirect.github.com/pylint-dev/pylint/issues/8701) - Discover `.pyi` files when linting. These can be ignored with the `ignore-patterns` setting. Closes [#​9097](https://redirect.github.com/pylint-dev/pylint/issues/9097) - Check `TypeAlias` and `TypeVar` (PEP 695) nodes for `invalid-name`. Refs [#​9196](https://redirect.github.com/pylint-dev/pylint/issues/9196) - Support for resolving external toml files named pylintrc.toml and .pylintrc.toml. Closes [#​9228](https://redirect.github.com/pylint-dev/pylint/issues/9228) - Check for `.clear`, `.discard`, `.pop` and `remove` methods being called on a set while it is being iterated over. Closes [#​9334](https://redirect.github.com/pylint-dev/pylint/issues/9334) ## New Checks - New message `use-yield-from` added to the refactoring checker. This message is emitted when yielding from a loop can be replaced by `yield from`. Closes [#​9229](https://redirect.github.com/pylint-dev/pylint/issues/9229). - Added a `deprecated-attribute` message to check deprecated attributes in the stdlib. Closes [#​8855](https://redirect.github.com/pylint-dev/pylint/issues/8855) ## False Positives Fixed - Fixed false positive for `inherit-non-class` for generic Protocols. Closes [#​9106](https://redirect.github.com/pylint-dev/pylint/issues/9106) - Exempt `TypedDict` from `typing_extensions` from `too-many-ancestor` checks. Refs [#​9167](https://redirect.github.com/pylint-dev/pylint/issues/9167) ## False Negatives Fixed - Extend broad-exception-raised and broad-exception-caught to except\*. Closes [#​8827](https://redirect.github.com/pylint-dev/pylint/issues/8827) - Fix a false-negative for unnecessary if blocks using a different than expected ordering of arguments. Closes [#​8947](https://redirect.github.com/pylint-dev/pylint/issues/8947). ## Other Bug Fixes - Improve the message provided for wrong-import-order check. Instead of the import statement ("import x"), the message now specifies the import that is out of order and which imports should come after it. As reported in the issue, this is particularly helpful if there are multiple imports on a single line that do not follow the PEP8 convention. The message will report imports as follows: For "import X", it will report "(standard/third party/first party/local) import X" For "import X.Y" and "from X import Y", it will report "(standard/third party/first party/local) import X.Y" The import category is specified to provide explanation as to why pylint has issued the message and guidence to the developer on how to fix the problem. Closes [#​8808](https://redirect.github.com/pylint-dev/pylint/issues/8808) ## Other Changes - Print how many files were checked in verbose mode. Closes [#​8935](https://redirect.github.com/pylint-dev/pylint/issues/8935) - Fix a crash when an enum class which is also decorated with a `dataclasses.dataclass` decorator is defined. Closes [#​9100](https://redirect.github.com/pylint-dev/pylint/issues/9100) ## Internal Changes - Update astroid version to 3.1.0. Refs [#​9457](https://redirect.github.com/pylint-dev/pylint/issues/9457) ### [`v3.0.4`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.0.4) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.0.3...v3.0.4) ## False Positives Fixed - `used-before-assignment` is no longer emitted when using a name in a loop and depending on an earlier name assignment in an `except` block paired with `else: continue`. Closes [#​6804](https://redirect.github.com/pylint-dev/pylint/issues/6804) - Avoid false positives for `no-member` involving function attributes supplied by decorators. Closes [#​9246](https://redirect.github.com/pylint-dev/pylint/issues/9246) - Fixed false positive nested-min-max for nested lists. Closes [#​9307](https://redirect.github.com/pylint-dev/pylint/issues/9307) - Fix false positive for `used-before-assignment` in a `finally` block when assignments took place in both the `try` block and each exception handler. Closes [#​9451](https://redirect.github.com/pylint-dev/pylint/issues/9451) ## Other Bug Fixes - Catch incorrect ValueError `"generator already executing"` for Python 3.12.0 - 3.12.2. This is fixed upstream in Python 3.12.3. Closes [#​9138](https://redirect.github.com/pylint-dev/pylint/issues/9138) ### [`v3.0.3`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.0.3) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.0.2...v3.0.3) ## What's new in Pylint 3.0.3? Release date: 2023-12-11 ## False Positives Fixed - Fixed false positive for `unnecessary-lambda` when the call has keyword arguments but not the lambda. Closes [#​9148](https://redirect.github.com/pylint-dev/pylint/issues/9148) - Fixed incorrect suggestion for shallow copy in unnecessary-comprehension Example of the suggestion: \#pylint: disable=missing-module-docstring a = \[1, 2, 3] b = \[x for x in a] b\[0] = 0 print(a) # \[1, 2, 3] After changing b = \[x for x in a] to b = a based on the suggestion, the script now prints \[0, 2, 3]. The correct suggestion should be use list(a) to preserve the original behavior. Closes [#​9172](https://redirect.github.com/pylint-dev/pylint/issues/9172) - Fix false positives for `undefined-variable` and `unused-argument` for classes and functions using Python 3.12 generic type syntax. Closes [#​9193](https://redirect.github.com/pylint-dev/pylint/issues/9193) - Fixed `pointless-string-statement` false positive for docstrings on Python 3.12 type aliases. Closes [#​9268](https://redirect.github.com/pylint-dev/pylint/issues/9268) - Fix false positive for `invalid-exception-operation` when concatenating tuples of exception types. Closes [#​9288](https://redirect.github.com/pylint-dev/pylint/issues/9288) ## Other Bug Fixes - Fix a bug where pylint was unable to walk recursively through a directory if the directory has an `__init__.py` file. Closes [#​9210](https://redirect.github.com/pylint-dev/pylint/issues/9210) ### [`v3.0.2`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.0.2) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v3.0.1...v3.0.2) ## False Positives Fixed - Fix `used-before-assignment` false positive for generic type syntax (PEP 695, Python 3.12). Closes [#​9110](https://redirect.github.com/pylint-dev/pylint/issues/9110) ## Other Bug Fixes - Escape special symbols and newlines in messages. Closes [#​7874](https://redirect.github.com/pylint-dev/pylint/issues/7874) - Fixes suggestion for `nested-min-max` for expressions with additive operators, list and dict comprehensions. Closes [#​8524](https://redirect.github.com/pylint-dev/pylint/issues/8524) - Fixes ignoring conditional imports with `ignore-imports=y`. Closes [#​8914](https://redirect.github.com/pylint-dev/pylint/issues/8914) - Emit `inconsistent-quotes` for f-strings with 3.12 interpreter only if targeting pre-3.12 versions. Closes [#​9113](https://redirect.github.com/pylint-dev/pylint/issues/9113) ### [`v3.0.1`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.0.1) ## False Positives Fixed - Fixed false positive for `inherit-non-class` for generic Protocols. Closes [#​9106](https://redirect.github.com/pylint-dev/pylint/issues/9106) ## Other Changes - Fix a crash when an enum class which is also decorated with a `dataclasses.dataclass` decorator is defined. Closes [#​9100](https://redirect.github.com/pylint-dev/pylint/issues/9100) ### [`v3.0.0`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v3.0.0) Pylint now support python 3.12 officially. This long anticipated major version also provides some important usability and performance improvements, along with enacting necessary breaking changes and long-announced deprecations. The documentation of each message with an example is very close too. The required astroid version is now 3.0.0. See the [astroid changelog](https://pylint.readthedocs.io/projects/astroid/en/latest/changelog.html#what-s-new-in-astroid-3-0-0) for additional fixes, features, and performance improvements applicable to pylint. Our code is now fully typed. The invalid-name message no longer checks for a minimum length of 3 characters by default. Dependencies like wrapt or setuptools were removed. A new json2 reporter has been added. It features an enriched output that is easier to parse and provides more info, here's a sample output. ```json { "messages": [ { "type": "convention", "symbol": "line-too-long", "message": "Line too long (1/2)", "messageId": "C0301", "confidence": "HIGH", "module": "0123", "obj": "", "line": 1, "column": 0, "endLine": 1, "endColumn": 4, "path": "0123", "absolutePath": "0123" } ], "statistics": { "messageTypeCount": { "fatal": 0, "error": 0, "warning": 0, "refactor": 0, "convention": 1, "info": 0 }, "modulesLinted": 1, "score": 5.0 } } ``` ## Breaking Changes - Enabling or disabling individual messages will now take effect even if an `--enable=all` or `disable=all` follows in the same configuration file (or on the command line). This means for the following example, `fixme` messages will now be emitted: ```bash pylint my_module --enable=fixme --disable=all ``` To regain the prior behavior, remove the superfluous earlier option. Closes [#​3696](https://redirect.github.com/pylint-dev/pylint/issues/3696) - Remove support for launching pylint with Python 3.7. Code that supports Python 3.7 can still be linted with the `--py-version=3.7` setting. Refs [#​6306](https://redirect.github.com/pylint-dev/pylint/issues/6306) - Disables placed in a `try` block now apply to the `except` block. Previously, they only happened to do so in the presence of an `else` clause. Refs [#​7767](https://redirect.github.com/pylint-dev/pylint/issues/7767) - `pyreverse` now uses a new default color palette that is more colorblind friendly. The color scheme is taken from `Paul Tol's Notes <https://personal.sron.nl/~pault/>`\_. If you prefer other colors, you can use the `--color-palette` option to specify custom colors. Closes [#​8251](https://redirect.github.com/pylint-dev/pylint/issues/8251) - Everything related to the `__implements__` construct was removed. It was based on PEP245 that was proposed in 2001 and rejected in 2006. The capability from pyreverse to take `__implements__` into account when generating diagrams was also removed. Refs [#​8404](https://redirect.github.com/pylint-dev/pylint/issues/8404) - `pyreverse`: Support for the `.vcg` output format (Visualization of Compiler Graphs) has been dropped. Closes [#​8416](https://redirect.github.com/pylint-dev/pylint/issues/8416) - The warning when the now useless old pylint cache directory (pylint.d) was found was removed. The cache dir is documented in `the FAQ <https://pylint.readthedocs.io/en/latest/faq.html#where-is-the-persistent-data-stored-to-compare-between-successive-runs>`\_. Refs [#​8462](https://redirect.github.com/pylint-dev/pylint/issues/8462) - Following a deprecation period, `pylint.config.PYLINTRC` was removed. Use the `pylint.config.find_default_config_files` generator instead. Closes [#​8862](https://redirect.github.com/pylint-dev/pylint/issues/8862) ## Changes requiring user actions - The `invalid-name` message no longer checks for a minimum length of 3 characters by default. (This was an unadvertised commingling of concerns between casing and name length, and users regularly reported this to be surprising.) If checking for a minimum length is still desired, it can be regained in two ways: - If you are content with a `disallowed-name` message (instead of `invalid-name`), then simply add the option `bad-names-rgxs="^..?$"`, which will fail 1-2 character-long names. (Ensure you enable `disallowed-name`.) - If you would prefer an `invalid-name` message to be emitted, or would prefer finer-grained control over the circumstances in which messages are emitted (classes vs. methods, etc.), then avail yourself of the regex options described `here <https://pylint.readthedocs.io/en/stable/user_guide/configuration/all-options.html#main-checker>`*. (In particular, take note of the commented out options in the "example configuration" given at the bottom of the section.) The prior regexes can be found in the `pull request <https://github.com/pylint-dev/pylint/pull/8813>`* that removed the length requirements. Closes [#​2018](https://redirect.github.com/pylint-dev/pylint/issues/2018) - The compare to empty string checker (`pylint.extensions.emptystring`) and the compare to zero checker (`pylint.extensions.compare-to-zero`) have been removed and their checks are now part of the implicit booleaness checker: - `compare-to-zero` was renamed `use-implicit-booleaness-not-comparison-to-zero` and `compare-to-empty-string` was renamed `use-implicit-booleaness-not-comparison-to-string` and they now need to be enabled explicitly. - The `pylint.extensions.emptystring` and `pylint.extensions.compare-to-zero` extensions no longer exist and need to be removed from the `load-plugins` option. - Messages related to implicit booleaness were made more explicit and actionable. This permits to make their likeness explicit and will provide better performance as they share most of their conditions to be raised. Closes [#​6871](https://redirect.github.com/pylint-dev/pylint/issues/6871) - epylint was removed. It now lives at: https://github.com/emacsorphanage/pylint. Refs [#​7737](https://redirect.github.com/pylint-dev/pylint/issues/7737) - The `overgeneral-exceptions` option now only takes fully qualified names into account (`builtins.Exception` not `Exception`). If you overrode this option, you need to use the fully qualified name now. There's still a warning, but it will be removed in 3.1.0. Refs [#​8411](https://redirect.github.com/pylint-dev/pylint/issues/8411) - Following a deprecation period, it's no longer possible to use `MASTER` or `master` as configuration section in `setup.cfg` or `tox.ini`. It's bad practice to not start a section title with the tool name. Please use `pylint.main` instead. Refs [#​8465](https://redirect.github.com/pylint-dev/pylint/issues/8465) - Package stats are now printed when running Pyreverse and a `--verbose` flag was added to get the original output with parsed modules. You might need to activate the verbose option if you want to keep the old output. Closes [#​8973](https://redirect.github.com/pylint-dev/pylint/issues/8973) ## New Features - A new `json2` reporter has been added. It features a more enriched output that is easier to parse and provides more info. Compared to `json` the only changes are that messages are now under the `"messages"` key and that `"message-id"` now follows the camelCase convention and is renamed to `"messageId"`. The new reporter also reports the "score" of the modules you linted as defined by the `evaluation` option and provides statistics about the modules you linted. We encourage users to use the new reporter as the `json` reporter will no longer be maintained. Closes [#​4741](https://redirect.github.com/pylint-dev/pylint/issues/4741) - In Pyreverse package dependency diagrams, show when a module imports another only for type-checking. Closes [#​8112](https://redirect.github.com/pylint-dev/pylint/issues/8112) - Add new option (`--show-stdlib`, `-L`) to `pyreverse`. This is similar to the behavior of `--show-builtin` in that standard library modules are now not included by default, and this option will include them. Closes [#​8181](https://redirect.github.com/pylint-dev/pylint/issues/8181) - Add Pyreverse option to exclude standalone nodes from diagrams with `--no-standalone`. Closes [#​8476](https://redirect.github.com/pylint-dev/pylint/issues/8476) ## New Checks - Added `DataclassChecker` module and `invalid-field-call` checker to check for invalid dataclasses.field() usage. Refs [#​5159](https://redirect.github.com/pylint-dev/pylint/issues/5159) - Add `return-in-finally` to emit a message if a return statement was found in a finally clause. Closes [#​8260](https://redirect.github.com/pylint-dev/pylint/issues/8260) - Add a new checker `kwarg-superseded-by-positional-arg` to warn when a function is called with a keyword argument which shares a name with a positional-only parameter and the function contains a keyword variadic parameter dictionary. It may be surprising behaviour when the keyword argument is added to the keyword variadic parameter dictionary. Closes [#​8558](https://redirect.github.com/pylint-dev/pylint/issues/8558) ## Extensions - Add new `prefer-typing-namedtuple` message to the `CodeStyleChecker` to suggest rewriting calls to `collections.namedtuple` as classes inheriting from `typing.NamedTuple` on Python 3.6+. Requires `load-plugins=pylint.extensions.code_style` and `enable=prefer-typing-namedtuple` to be raised. Closes [#​8660](https://redirect.github.com/pylint-dev/pylint/issues/8660) ## False Positives Fixed - Extend concept of "function ambiguity" in `safe_infer()` from differing number of function arguments to differing set of argument names. Solves false positives in `tensorflow`. Closes [#​3613](https://redirect.github.com/pylint-dev/pylint/issues/3613) - Fix `unused-argument` false positive when `__new__` does not use all the arguments of `__init__`. Closes [#​3670](https://redirect.github.com/pylint-dev/pylint/issues/3670) - Fix a false positive for `invalid-name` when a type-annotated class variable in an `enum.Enum` class has no assigned value. Refs [#​7402](https://redirect.github.com/pylint-dev/pylint/issues/7402) - Fix `unused-import` false positive for usage of `six.with_metaclass`. Closes [#​7506](https://redirect.github.com/pylint-dev/pylint/issues/7506) - Fix false negatives and false positives for `too-many-try-statements`, `too-complex`, and `too-many-branches` by correctly counting statements under a `try`. Refs [#​7767](https://redirect.github.com/pylint-dev/pylint/issues/7767) - When checking for unbalanced dict unpacking in for-loops, Pylint will now test whether the length of each value to be unpacked matches the number of unpacking targets. Previously, Pylint would test the number of values for the loop iteration, which would produce a false unbalanced-dict-unpacking warning. Closes [#​8156](https://redirect.github.com/pylint-dev/pylint/issues/8156) - Fix false positive for `used-before-assignment` when usage and assignment are guarded by the same test in different statements. Closes [#​8167](https://redirect.github.com/pylint-dev/pylint/issues/8167) - Adds `asyncSetUp` to the default `defining-attr-methods` list to silence `attribute-defined-outside-init` warning when using `unittest.IsolatedAsyncioTestCase`. Refs [#​8403](https://redirect.github.com/pylint-dev/pylint/issues/8403) - `logging-not-lazy` is not longer emitted for explicitly concatenated string arguments. Closes [#​8410](https://redirect.github.com/pylint-dev/pylint/issues/8410) - Fix false positive for isinstance-second-argument-not-valid-type when union types contains None. Closes [#​8424](https://redirect.github.com/pylint-dev/pylint/issues/8424) - `invalid-name` now allows for integers in `typealias` names: - now valid: `Good2Name`, `GoodName2`. - still invalid: `_1BadName`. Closes [#​8485](https://redirect.github.com/pylint-dev/pylint/issues/8485) - No longer consider `Union` as type annotation as type alias for naming checks. Closes [#​8487](https://redirect.github.com/pylint-dev/pylint/issues/8487) - `unnecessary-lambda` no longer warns on lambdas which use its parameters in their body (other than the final arguments), e.g. `lambda foo: (bar if foo else baz)(foo)`. Closes [#​8496](https://redirect.github.com/pylint-dev/pylint/issues/8496) - Fixed `unused-import` so that it observes the `dummy-variables-rgx` option. Closes [#​8500](https://redirect.github.com/pylint-dev/pylint/issues/8500) - `Union` typed variables without assignment are no longer treated as `TypeAlias`. Closes [#​8540](https://redirect.github.com/pylint-dev/pylint/issues/8540) - Allow parenthesized implicitly concatenated strings when `check-str-concat-over-line-jumps` is enabled. Closes [#​8552](https://redirect.github.com/pylint-dev/pylint/issues/8552). - Fix false positive for `positional-only-arguments-expected` when a function contains both a positional-only parameter that has a default value, and `**kwargs`. Closes [#​8555](https://redirect.github.com/pylint-dev/pylint/issues/8555) - Fix false positive for `keyword-arg-before-vararg` when a positional-only parameter with a default value precedes `*args`. Closes [#​8570](https://redirect.github.com/pylint-dev/pylint/issues/8570) - Fix false positive for `arguments-differ` when overriding `__init_subclass__`. Closes [#​8919](https://redirect.github.com/pylint-dev/pylint/issues/8919) - Fix a false positive for `no-value-for-parameter` when a staticmethod is called in a class body. Closes [#​9036](https://redirect.github.com/pylint-dev/pylint/issues/9036) ## False Negatives Fixed - Emit `used-before-assignment` when calling module-level functions before definition. Closes [#​1144](https://redirect.github.com/pylint-dev/pylint/issues/1144) - Apply `infer_kwarg_from_call()` to more checks These mostly solve false negatives for various checks, save for one false positive for `use-maxsplit-arg`. Closes [#​7761](https://redirect.github.com/pylint-dev/pylint/issues/7761) - `TypeAlias` variables defined in functions are now checked for `invalid-name` errors. Closes [#​8536](https://redirect.github.com/pylint-dev/pylint/issues/8536) - Fix false negative for `no-value-for-parameter` when a function, whose signature contains both a positional-only parameter `name` and also `*kwargs`, is called with a keyword-argument for `name`. Closes [#​8559](https://redirect.github.com/pylint-dev/pylint/issues/8559) - Fix a false negative for `too-many-arguments` by considering positional-only and keyword-only parameters. Closes [#​8667](https://redirect.github.com/pylint-dev/pylint/issues/8667) - Emit `assignment-from-no-return` for calls to builtin methods like `dict.update()`. Calls to `list.sort()` now raise `assignment-from-no-return` rather than `assignment-from-none` for consistency. Closes [#​8714](https://redirect.github.com/pylint-dev/pylint/issues/8714) Closes [#​8810](https://redirect.github.com/pylint-dev/pylint/issues/8810) - `consider-using-augmented-assign` is now applied to dicts and lists as well. Closes [#​8959](https://redirect.github.com/pylint-dev/pylint/issues/8959) ## Other Bug Fixes - Support `duplicate-code` message when parallelizing with `--jobs`. Closes [#​374](https://redirect.github.com/pylint-dev/pylint/issues/374) - Support `cyclic-import` message when parallelizing with `--jobs`. Closes [#​4171](https://redirect.github.com/pylint-dev/pylint/issues/4171) - `--jobs` can now be used with `--load-plugins`. This had regressed in astroid 2.5.0. Closes [#​4874](https://redirect.github.com/pylint-dev/pylint/issues/4874) - docparams extension considers type comments as type documentation. Closes [#​6287](https://redirect.github.com/pylint-dev/pylint/issues/6287) - When parsing comma-separated lists of regular expressions in the config, ignore commas that are inside braces since those indicate quantifiers, not delineation between expressions. Closes [#​7229](https://redirect.github.com/pylint-dev/pylint/issues/7229) - The `ignored-modules` option will now be correctly taken into account for `no-name-in-module`. Closes [#​7578](https://redirect.github.com/pylint-dev/pylint/issues/7578) - `sys.argv` is now always correctly considered as impossible to infer (instead of using the actual values given to pylint). Closes [#​7710](https://redirect.github.com/pylint-dev/pylint/issues/7710) - Avoid duplicative warnings for unqualified exception names in the `overgeneral-exceptions` setting when running with `--jobs`. Closes [#​7774](https://redirect.github.com/pylint-dev/pylint/issues/7774) - Don't show class fields more than once in Pyreverse diagrams. Closes [#​8189](https://redirect.github.com/pylint-dev/pylint/issues/8189) - Fix `used-before-assignment` false negative when TYPE_CHECKING imports are used in multiple scopes. Closes [#​8198](https://redirect.github.com/pylint-dev/pylint/issues/8198) - `--clear-cache-post-run` now also clears LRU caches for pylint utilities holding references to AST nodes. Closes [#​8361](https://redirect.github.com/pylint-dev/pylint/issues/8361) - Fix a crash when `TYPE_CHECKING` is used without importing it. Closes [#​8434](https://redirect.github.com/pylint-dev/pylint/issues/8434) - Fix a `used-before-assignment` false positive when imports are made under the `TYPE_CHECKING` else if branch. Closes [#​8437](https://redirect.github.com/pylint-dev/pylint/issues/8437) - Fix a regression of `preferred-modules` where a partial match was used instead of the required full match. Closes [#​8453](https://redirect.github.com/pylint-dev/pylint/issues/8453) - Fix a crash in pyreverse when "/" characters are used in the output filename e.g pyreverse -o png -p name/ path/to/project. Closes [#​8504](https://redirect.github.com/pylint-dev/pylint/issues/8504) - Don't show arrows more than once in Pyreverse diagrams. Closes [#​8522](https://redirect.github.com/pylint-dev/pylint/issues/8522) - Improve output of `consider-using-generator` message for `min()` calls with `default` keyword. Closes [#​8563](https://redirect.github.com/pylint-dev/pylint/issues/8563) - Fixed a crash when generating a configuration file: `tomlkit.exceptions.TOMLKitError: Can't add a table to a dotted key` caused by tomlkit `v0.11.8`. Closes [#​8632](https://redirect.github.com/pylint-dev/pylint/issues/8632) - Fix a line break error in Pyreverse dot output. Closes [#​8671](https://redirect.github.com/pylint-dev/pylint/issues/8671) - Fix a false positive for `method-hidden` when using `cached_property` decorator. Closes [#​8753](https://redirect.github.com/pylint-dev/pylint/issues/8753) - Dunder methods defined in lambda do not trigger `unnecessary-dunder-call` anymore, if they cannot be replaced by the non-dunder call. Closes [#​8769](https://redirect.github.com/pylint-dev/pylint/issues/8769) - Don't show duplicate type annotations in Pyreverse diagrams. Closes [#​8888](https://redirect.github.com/pylint-dev/pylint/issues/8888) - Fixing inconsistent hashing issue in `BaseChecker` causing some reports not being exported. Closes [#​9001](https://redirect.github.com/pylint-dev/pylint/issues/9001) - Don't add `Optional` to `|` annotations with `None` in Pyreverse diagrams. Closes [#​9014](https://redirect.github.com/pylint-dev/pylint/issues/9014) - Pyreverse doesn't show multiple class association arrows anymore, but only the strongest one. Refs [#​9045](https://redirect.github.com/pylint-dev/pylint/issues/9045) - Prevented data loss in the linter stats for messages relating to the linter itself (e.g. `unknown-option-value`), fixing problems with score, fail-on, etc. Closes [#​9059](https://redirect.github.com/pylint-dev/pylint/issues/9059) - Fix crash in refactoring checker when unary operand used with variable in for loop. Closes [#​9074](https://redirect.github.com/pylint-dev/pylint/issues/9074) ## Other Changes - Pylint now exposes its type annotations. Closes [#​5488](https://redirect.github.com/pylint-dev/pylint/issues/5488) and [#​2079](https://redirect.github.com/pylint-dev/pylint/issues/2079) - Search for `pyproject.toml` recursively in parent directories up to a project or file system root. Refs [#​7163](https://redirect.github.com/pylint-dev/pylint/issues/7163), Closes [#​3289](https://redirect.github.com/pylint-dev/pylint/issues/3289) - All code related to the optparse config parsing has been removed. Refs [#​8405](https://redirect.github.com/pylint-dev/pylint/issues/8405) - Pylint now supports python 3.12. Refs [#​8718](https://redirect.github.com/pylint-dev/pylint/issues/8718) - Add a CITATION.cff file to the root of the repository containing the necessary metadata to cite Pylint. Closes [#​8760](https://redirect.github.com/pylint-dev/pylint/issues/8760) - Renamed the "unneeded-not" error into "unnecessary_negation" to be clearer. Closes [#​8789](https://redirect.github.com/pylint-dev/pylint/issues/8789) ## Internal Changes - `get_message_definition` was removed from the base checker API. You can access message definitions through the `MessageStore`. Refs [#​8401](https://redirect.github.com/pylint-dev/pylint/issues/8401) - Everything related to the `__implements__` construct was removed. It was based on PEP245 that was proposed in 2001 and rejected in 2006. All the classes inheriting `Interface` in `pylint.interfaces` were removed. `Checker` should only inherit `BaseChecker` or any of the other checker types from `pylint.checkers`. `Reporter` should only inherit `BaseReporter`. Refs [#​8404](https://redirect.github.com/pylint-dev/pylint/issues/8404) - `modname` and `msg_store` are now required to be given in `FileState`. `collect_block_lines` has also been removed. `Pylinter.current_name` cannot be null anymore. Refs [#​8407](https://redirect.github.com/pylint-dev/pylint/issues/8407) - `Reporter.set_output` was removed in favor of `reporter.out = stream`. Refs [#​8408](https://redirect.github.com/pylint-dev/pylint/issues/8408) - A number of old utility functions and classes have been removed: `MapReduceMixin`: To make a checker reduce map data simply implement `get_map_data` and `reduce_map_data`. `is_inside_lambda`: Use `utils.get_node_first_ancestor_of_type(x, nodes.Lambda)` `check_messages`: Use `utils.only_required_for_messages` `is_class_subscriptable_pep585_with_postponed_evaluation_enabled`: Use `is_postponed_evaluation_enabled(node)` and `is_node_in_type_annotation_context(node)` `get_python_path`: assumption that there's always an **init**.py is not true since python 3.3 and is causing problems, particularly with PEP 420. Use `discover_package_path` and pass source root(s). `fix_import_path`: Use `augmented_sys_path` and pass additional `sys.path` entries as an argument obtained from `discover_package_path`. `get_global_option`: Use `checker.linter.config` to get all global options. Related private objects have been removed as well. Refs [#​8409](https://redirect.github.com/pylint-dev/pylint/issues/8409) - `colorize_ansi` now only accepts a `MessageStyle` object. Refs [#​8412](https://redirect.github.com/pylint-dev/pylint/issues/8412) - Following a deprecation period, `Pylinter.check` now only works with sequences of strings, not strings. Refs [#​8463](https://redirect.github.com/pylint-dev/pylint/issues/8463) - Following a deprecation period, `ColorizedTextReporter` only accepts `ColorMappingDict`. Refs [#​8464](https://redirect.github.com/pylint-dev/pylint/issues/8464) - Following a deprecation period, `MessageTest`'s `end_line` and `end_col_offset` must be accurate in functional tests (for python 3.8 or above on cpython, and for python 3.9 or superior on pypy). Refs [#​8466](https://redirect.github.com/pylint-dev/pylint/issues/8466) - Following a deprecation period, the `do_exit` argument of the `Run` class (and of the `_Run` class in testutils) were removed. Refs [#​8472](https://redirect.github.com/pylint-dev/pylint/issues/8472) - Following a deprecation period, the `py_version` argument of the `MessageDefinition.may_be_emitted` function is now required. The most likely solution is to use 'linter.config.py_version' if you need to keep using this function, or to use 'MessageDefinition.is_message_enabled' instead. Refs [#​8473](https://redirect.github.com/pylint-dev/pylint/issues/8473) - Following a deprecation period, the `OutputLine` class now requires the right number of argument all the time. The functional output can be regenerated automatically to achieve that easily. Refs [#​8474](https://redirect.github.com/pylint-dev/pylint/issues/8474) - Following a deprecation period, `is_typing_guard`, `is_node_in_typing_guarded_import_block` and `is_node_in_guarded_import_block` from `pylint.utils` were removed: use a combination of `is_sys_guard` and `in_type_checking_block` instead. Refs [#​8475](https://redirect.github.com/pylint-dev/pylint/issues/8475) - Following a deprecation period, the `location` argument of the `Message` class must now be a `MessageLocationTuple`. Refs [#​8477](https://redirect.github.com/pylint-dev/pylint/issues/8477) - Following a deprecation period, the `check_single_file` function of the `Pylinter` is replaced by `Pylinter.check_single_file_item`. Refs [#​8478](https://redirect.github.com/pylint-dev/pylint/issues/8478) ## Performance Improvements - `pylint` runs (at least) ~5% faster after improvements to `astroid` that make better use of the inference cache. Refs [pylint-dev/astroid#529](https://redirect.github.com/pylint-dev/astroid/issues/529) - - Optimize `is_trailing_comma()`. - Cache `class_is_abstract()`. Refs [#​1954](https://redirect.github.com/pylint-dev/pylint/issues/1954) - Exit immediately if all messages are disabled. Closes [#​8715](https://redirect.github.com/pylint-dev/pylint/issues/8715) ### [`v2.17.7`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v2.17.7) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v2.17.6...v2.17.7) 2.17.7 is the last release before we only support pylint 3.0.0 or superior and python 3.8 or superior. ## False Positives Fixed - Fix a regression in pylint 2.17.6 / astroid 2.15.7 causing various messages for code involving `TypeVar`. Closes [#​9069](https://redirect.github.com/pylint-dev/pylint/issues/9069) ## Other Bug Fixes - Fix crash in refactoring checker when unary operand used with variable in for loop. Closes [#​9074](https://redirect.github.com/pylint-dev/pylint/issues/9074) ### [`v2.17.6`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v2.17.6) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v2.17.5...v2.17.6) ## Other Bug Fixes - When parsing comma-separated lists of regular expressions in the config, ignore commas that are inside braces since those indicate quantifiers, not delineation between expressions. Closes [#​7229](https://redirect.github.com/pylint-dev/pylint/issues/7229) - `sys.argv` is now always correctly considered as impossible to infer (instead of using the actual values given to pylint). Closes [#​9047](https://redirect.github.com/pylint-dev/pylint/issues/9047) - Don't show class fields more than once in Pyreverse diagrams. Closes [#​8189](https://redirect.github.com/pylint-dev/pylint/issues/8189) - Don't show arrows more than once in Pyreverse diagrams. Closes [#​8522](https://redirect.github.com/pylint-dev/pylint/issues/8522) - Don't show duplicate type annotations in Pyreverse diagrams. Closes [#​8888](https://redirect.github.com/pylint-dev/pylint/issues/8888) - Don't add `Optional` to `|` annotations with `None` in Pyreverse diagrams. Closes [#​9014](https://redirect.github.com/pylint-dev/pylint/issues/9014) ### [`v2.17.5`](https://redirect.github.com/pylint-dev/pylint/releases/tag/v2.17.5) [Compare Source](https://redirect.github.com/pylint-dev/pylint/compare/v2.17.4...v2.17.5) ## What's new in Pylint 2.17.5? Release date: 2023-07-26 ## False Positives Fixed - Fix a false positive for `unused-variable` when there is an import in a `if TYPE_CHECKING:` block and `allow-global-unused-variables` is set to `no` in the configuration. Closes [#​8696](https://redirect.github.com/pylint-dev/pylint/issues/8696) - Fix false positives generated when supplying arguments as `**kwargs` to IO calls like open(). Closes [#​8719](https://redirect.github.com/pylint-dev/pylint/issues/8719) - Fix a false positive where pylint was ignoring method calls annotated as `NoReturn` during the `inconsistent-return-statements` check. Closes [#​8747](https://redirect.github.com/pylint-dev/pylint/issues/8747) - Exempt parents with only </details> --- ### Configuration π **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). π¦ **Automerge**: Disabled by config. Please merge this manually once you are satisfied. β» **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. π **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/equinix/ansible-collection-equinix). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41OS4yIiwidXBkYXRlZEluVmVyIjoiMzguODAuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->
Type of Changes
Description
Closes #2018
Many users have expressed surprise that
invalid-name
, which styles itself as a casing checker for snake case, camel case, etc., also checks for a minimum of three characters. We could clarify in the message, we could decouple the two, or we could build a new feature. This PR takes the middle ground: decoupling length from casing and then showing users who still want to check for length some alternatives. The simplest alternative involves adding a single setting.