Skip to content

Conversation

@d3rky
Copy link
Contributor

@d3rky d3rky commented Jan 19, 2026

Closes MPT-17125

Release Notes

  • Restructured pyproject.toml flake8 plugin configuration from flat top-level keys to nested sections under [tool.ruff.lint.*]
  • Updated flake8-import-conventions.banned-from to include both "ast" and "datetime"
  • Organized lint rules into dedicated nested sections:
    • [tool.ruff.lint.flake8-import-conventions] with banned-from and aliases
    • [tool.ruff.lint.flake8-quotes] with inline-quotes configuration
    • [tool.ruff.lint.mccabe] with max-complexity setting
    • [tool.ruff.lint.pydocstyle] with convention configuration
  • Added [tool.ruff.lint.per-file-ignores] to preserve per-file ignore rules for tests
  • Migrates configuration to structured Ruff format to resolve dependabot compatibility issues

@d3rky d3rky requested a review from a team as a code owner January 19, 2026 17:47
@d3rky d3rky requested review from jentyk and svazquezco January 19, 2026 17:47
@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

📝 Walkthrough

Walkthrough

Configuration migration in pyproject.toml reorganizes flake8-style top-level plugin keys into nested Ruff lint sections, consolidating import conventions, quotes, mccabe, and pydocstyle settings while adding "ast" to the banned-from list.

Changes

Cohort / File(s) Summary
Configuration Migration
pyproject.toml
Migrates plugin configuration from flat top-level keys under [tool.flake8] to nested [tool.ruff.lint.*] sections. Reorganizes flake8-import-conventions (adds "ast" to banned-from), flake8-quotes, mccabe, and pydocstyle settings into dedicated subsections. Introduces [tool.ruff.lint.per-file-ignores] for test file exceptions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Jira Issue Key In Title ✅ Passed The PR title contains exactly one Jira issue key in the correct format. The issue key 'MPT-17125' is present at the beginning of the title, which follows the required MPT-XXXX pattern.
Test Coverage Required ✅ Passed PR adds 301 test files alongside code files, with modifications limited to pyproject.toml configuration.
Single Commit Required ✅ Passed The PR contains exactly one commit (052d03f), meeting the single commit requirement for clean git history.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@d3rky
Copy link
Contributor Author

d3rky commented Jan 19, 2026

@sonarqubecloud
Copy link

@d3rky d3rky merged commit e4512b3 into main Jan 19, 2026
4 checks passed
@d3rky d3rky deleted the bugfix/MPT-17125/fix-pyproject-for-dependabot branch January 19, 2026 18:03
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.

3 participants