Skip to content

Removed unnecessary Pylint flags#535

Merged
phiwuu merged 1 commit intomainfrom
internal/pylint-cleanup
Feb 18, 2026
Merged

Removed unnecessary Pylint flags#535
phiwuu merged 1 commit intomainfrom
internal/pylint-cleanup

Conversation

@phiwuu
Copy link
Member

@phiwuu phiwuu commented Feb 18, 2026

  • Removed locally-disabled:
    The flag locally-disabled is disabled by default, and can only
    be enabled by adding it to the enable section. But the flag was
    listed in the disable section, which is useless and only bloats
    up the Pylint configuration file.
  • Other flags were removed because there are no issues with the
    code with respect to those flags.

@phiwuu phiwuu requested a review from a team as a code owner February 18, 2026 03:16
@phiwuu phiwuu added the internal Affects the CI, tests or refactorings only, not relevant to the end-user label Feb 18, 2026
- Removed `locally-disabled`:
  The flag `locally-disabled` is disabled by default, and can only
  be enabled by adding it to the `enable` section. But the flag was
  listed in the `disable` section, which is useless and only bloats
  up the Pylint configuration file.
- Other flags were removed because there are no issues with the
  code with respect to those flags.
@phiwuu phiwuu force-pushed the internal/pylint-cleanup branch from bd03dcd to 7ce41ad Compare February 18, 2026 03:18
@phiwuu phiwuu enabled auto-merge (squash) February 18, 2026 03:47
@phiwuu phiwuu merged commit 79dfe7f into main Feb 18, 2026
23 checks passed
@phiwuu phiwuu deleted the internal/pylint-cleanup branch February 18, 2026 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Affects the CI, tests or refactorings only, not relevant to the end-user

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants