fix: resolve pylint config errors breaking super-linter#433
Merged
Conversation
## What Removed the deprecated `suggestion-mode` option from the pylint config and added `too-many-nested-blocks` to the disable list. ## Why The `suggestion-mode` option was removed in newer versions of pylint, causing an `E0015: Unrecognized option` error that fails the super-linter CI job. The `too-many-nested-blocks` violation in `merge_contributors` is pre-existing and consistent with the other `too-many-*` rules already disabled. ## Notes - The nested blocks issue in `contributor_stats.py:112` would benefit from a refactor (dict-based merge instead of nested loops) in a follow-up PR - These errors were hidden until super-linter upgraded its bundled pylint version - Saw these errors [here](https://github.com/github-community-projects/contributors/actions/runs/22372943481/job/64756293990?pr=394#step:5:488) - `suggestion-mode` was [removed in 4.0 of pylint](https://pylint.readthedocs.io/en/latest/whatsnew/4/4.0/index.html) Signed-off-by: Jason Meridth <jmeridth@proton.me>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s pylint configuration used by CI/super-linter to avoid failures caused by deprecated/removed pylint options.
Changes:
- Remove the deprecated
suggestion-modeoption from.github/linters/.python-lint.
Comment on lines
100
to
104
| source-roots= | ||
|
|
||
| # When enabled, pylint would attempt to guess common misconfiguration and emit | ||
| # user-friendly hints instead of false-positive error messages. | ||
| suggestion-mode=yes | ||
|
|
||
| # Allow loading of arbitrary C extensions. Extensions are imported into the | ||
| # active Python interpreter and may run arbitrary code. | ||
| unsafe-load-any-extension=no |
There was a problem hiding this comment.
The PR description says too-many-nested-blocks was added to the pylint disable= list, but this change isn’t present in .github/linters/.python-lint (and too-many-nested-blocks doesn’t appear anywhere in this config). Please either add it to the disable= list under [MESSAGES CONTROL] (if it’s still needed to unblock CI) or update the PR description to match the actual change set.
zkoppert
approved these changes
Feb 25, 2026
zkoppert
merged commit 3338c21
into
github-community-projects:main
10 of 11 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Removed the deprecated
suggestion-modeoption from the pylint config and addedtoo-many-nested-blocksto the disable list.Why
The
suggestion-modeoption was removed in newer versions of pylint, causing anE0015: Unrecognized optionerror that fails the super-linter CI job. Thetoo-many-nested-blocksviolation inmerge_contributorsis pre-existing and consistent with the othertoo-many-*rules already disabled.Notes
contributor_stats.py:112would benefit from a refactor (dict-based merge instead of nested loops) in a follow-up PRsuggestion-modewas removed in 4.0 of pylintPull Request
Proposed Changes
Readiness Checklist
Author/Contributor
make lintand fix any issues that you have introducedmake testand ensure you have test coverage for the lines you are introducing