Skip to content

fix: update pylint config format for pyproject-fmt compatibility#1281

Merged
adamtheturtle merged 1 commit intomainfrom
fix/ci-pylint-config
Mar 6, 2026
Merged

fix: update pylint config format for pyproject-fmt compatibility#1281
adamtheturtle merged 1 commit intomainfrom
fix/ci-pylint-config

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Mar 6, 2026

Fixes CI lint failures on main. Change MESSAGES CONTROL to MESSAGES-CONTROL for pyproject-fmt compatibility.

Made with Cursor


Note

Low Risk
Low risk: only reformats pyproject.toml Pylint configuration keys (quoting/section naming) to satisfy pyproject-fmt and fix CI lint parsing, with no runtime code changes.

Overview
Fixes CI lint failures by updating the [tool.pylint] configuration in pyproject.toml to a pyproject-fmt-compatible key format.

This removes quoted section prefixes (e.g., "MASTER".* -> MASTER.*) and renames the Pylint section key from MESSAGES CONTROL to MESSAGES-CONTROL while keeping the enabled/disabled message lists and other settings unchanged.

Written by Cursor Bugbot for commit 659e9de. This will update automatically on new commits. Configure here.

Change MESSAGES CONTROL to MESSAGES-CONTROL to fix pre-commit lint failures on main.

Made-with: Cursor
@adamtheturtle adamtheturtle enabled auto-merge (squash) March 6, 2026 07:52
@adamtheturtle adamtheturtle disabled auto-merge March 6, 2026 07:53
@adamtheturtle adamtheturtle merged commit 7178eca into main Mar 6, 2026
13 of 29 checks passed
@adamtheturtle adamtheturtle deleted the fix/ci-pylint-config branch March 6, 2026 07:53
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

# multiple time (only on the command line, not in the configuration file where
# it should appear only once). See also the "--disable" option for examples.
"MESSAGES CONTROL".enable = [
MESSAGES-CONTROL.enable = [
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pylint silently ignores renamed MESSAGES-CONTROL section

High Severity

The change from "MESSAGES CONTROL" (quoted key with a space) to MESSAGES-CONTROL (bare key with a hyphen) alters the actual TOML key name. These are semantically different keys. Pylint recognizes MESSAGES CONTROL (space) or MESSAGES_CONTROL (underscore) as valid section names, but MESSAGES-CONTROL (hyphen) is not a recognized section name and will be silently ignored. This means both the enable and disable message lists will have no effect, potentially causing unexpected lint failures or missing warnings.

Additional Locations (1)

Fix in Cursor Fix in Web

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