Skip to content

[codex] Make useless Pylint suppressions fail lint#3019

Merged
adamtheturtle merged 1 commit into
mainfrom
codex/pylint-useless-suppression-error-20260520
May 20, 2026
Merged

[codex] Make useless Pylint suppressions fail lint#3019
adamtheturtle merged 1 commit into
mainfrom
codex/pylint-useless-suppression-error-20260520

Conversation

@adamtheturtle
Copy link
Copy Markdown
Member

@adamtheturtle adamtheturtle commented May 20, 2026

Summary

  • Configure Pylint to fail when useless-suppression is emitted.
  • Keep existing useless-suppression enablement and project-specific Pylint settings intact.

Validation

  • Parsed pyproject.toml with tomllib.
  • Ran pylint --rcfile pyproject.toml --list-msgs-enabled.
  • Ran repository commit hooks for the changed file.

Note

Low Risk
Low risk configuration-only change that may cause CI/lint to start failing if the repo contains outdated # pylint: disable pragmas that no longer suppress anything.

Overview
Updates pyproject.toml to make Pylint exit non-zero when it emits useless-suppression (via MAIN.fail-on). This turns previously non-fatal useless suppression warnings into hard lint failures, forcing removal of redundant pylint disable comments.

Reviewed by Cursor Bugbot for commit 1fe5f04. Bugbot is set up for automated code reviews on this repo. Configure here.

@adamtheturtle adamtheturtle marked this pull request as ready for review May 20, 2026 12:08
@adamtheturtle adamtheturtle merged commit 742d6a2 into main May 20, 2026
15 checks passed
@adamtheturtle adamtheturtle deleted the codex/pylint-useless-suppression-error-20260520 branch May 20, 2026 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant