← Back to issue list

lint: remove pylint useless suppression "too-many-lines"

View original Github issue

Metadata

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

Current evaluation

Merged in a8d5fc5: removed the obsolete pylint too-many-lines suppression after the file fell below 1,000 LOC.

Resolution: closed resolved

Reason: PR #216 was approved and merged as commit a8d5fc5, deleting four suppression-related lines without functional changes.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- I recently brought a file under 1,000 LOC, so the pylint suppression can be removed.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in a8d5fc5: removed the obsolete pylint too-many-lines suppression after the file fell below 1,000 LOC.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR removing the pylint too-many-lines suppression after a file was reduced below 1,000 lines of code.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#3813 lint: bump pylint snapcraft merged Merged in commit 522b301a9: updated Pylint and removed a redundant too-many-lines suppression across three files.
85%
#291 lint: remove leftover pylint directive craft-parts merged Merged PR #291 in commit fb583a60, removing one leftover pylint directive from a single file.
80%
#192 lint: enable pylint useless-supression, minor reformatting craft-providers merged Merged lint cleanup in craft-providers#192 via commit f644c25, enabling pylint's useless-suppression check, removing obsolete suppressions, and reformatting seven files without functional changes.
75%
#4817 style: remove pylint snapcraft merged Merged as a9badaa32, removing Pylint from lint tooling, configuration, and source suppressions after Ruff had replaced nearly all of it.
75%
#66 lint: update pylint configuration craft-store merged Merged PR #66 as commit 20c7aa9, refreshing the pylint configuration without functional code changes.
74%