← Back to issue list

lint: move .pylintrc to pyproject.toml

View original Github issue

Metadata

Project
craft-providers
Number
#128
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #128 (commit 7067802) by mr-cal moved pylint config from .pylintrc into pyproject.toml, matching snapcraft/craft-parts style; net +16/-524 lines, 2 files.

Resolution: closed resolved

Reason: The PR was approved and merged by a maintainer, landing as commit 7067802 "lint: move .pylintrc to pyproject.toml". File history for .pylintrc shows 7067802 as the removing commit, and no .pylintrc exists at HEAD. The sole review comment was resolved in-PR, with accidental value changes (max-locals 16->18, min-similar-lines 12->13) reverted by the author.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- Follow `pylint` config style used in `snapcraft` and `craft-parts`. I prefer this for a few reasons: 1. Consolidates multiple configs into one place 2. Marginally less prone to breakage 3. Easier to work with (19 vs 524 lines of code)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #128 (commit 7067802) by mr-cal moved pylint config from .pylintrc into pyproject.toml, matching snapcraft/craft-parts style; net +16/-524 lines, 2 files.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR moving pylint configuration from .pylintrc to pyproject.toml to align with snapcraft and craft-parts conventions.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#127 lint: update pylint to 2.14 craft-providers merged Merged PR #127 via commit a38cb5a, updating .pylintrc for pylint 2.14 compatibility in change 69a2c0e.
72%