tests: configure pylint to issue fixme-info
Metadata
Current evaluation
Merged #11 via commit 08b51a7c, configuring pylint-fixme-info to report TODO/FIXME messages as informational and removing pylint's --fail-under CI check.
Resolution: closed resolved
Reason: PR #11 was merged with commit 08b51a7c, implementing the requested pylint plugin configuration and CI change.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Use the pylint_fixme_info plugin to make TODO/FIXME messages informational
rather than warnings or errors. This allows us to remove the --fail-under
switch when running pylint on CI.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #11 via commit 08b51a7c, configuring pylint-fixme-info to report TODO/FIXME messages as informational and removing pylint's --fail-under CI check. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/craft-parts#11 via 08b51a7c: pylint now emits TODO/FIXME as informational messages, and CI no longer needs the --fail-under threshold. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Configured pylint to treat TODO and FIXME messages as informational instead of warnings or errors, removing the --fail-under CI flag. Approved by two reviewers and passed all checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged updates configuring pylint to treat TODO/FIXME messages as informational via the pylint_fixme_info plugin, eliminating the --fail-under CI switch requirement. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-parts#166
(confidence 90%)
Later merged PR re-enabled pylint fixme-info reporting after lint configuration changed.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #166 linters: re-enable pylint fixme info | craft-parts | merged | Merged PR #166 in commit 86dc5757, re-enabling pylint's fixme-info reporting after maintainer approval. | |
| #29 tests: address pylint issues | craft-parts | merged | Merged: small test-only cleanup of Pylint warnings (3 files, +7/-4) by maintainer cmatsuoka. Landed as commit 0a3d2554 via merge commit efb9bcc3 (PR #29). | |
| #112 static: bump pylint to latest version | craft-providers | merged | Merged #112 as bd02f17 (change b07e4f7), updating pylint to support pylint-fixme-info v1.0.3. | |
| #57 tests: update pylint and address new warnings | craft-parts | merged | Merged PR #57 in commit 78560246 upgraded pylint to 2.8.2 and addressed the resulting warnings with minor test changes. | |
| #66 lint: update pylint configuration | craft-store | merged | Merged PR #66 as commit 20c7aa9, refreshing the pylint configuration without functional code changes. |