tests: configure pylint to issue fixme-info
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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 issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #166 linters: re-enable pylint fixme info | craft-parts | merged | Merged a one-line change to re-enable pylint fixme info in the linters configuration. Approved by one reviewer, passed all CI checks, and resolved with zero outstanding comments. | |
| #57 tests: update pylint and address new warnings | craft-parts | merged | Merged after two approvals and passing CI. Updated pylint to 2.8.2 and resolved newly introduced linting warnings in the test suite with minor code adjustments across a single file. | |
| #29 tests: address pylint issues | craft-parts | merged | Merged to address pylint warnings in test files. Approved by three reviewers and passed CI. The update modified three files, adding seven and removing four lines. | |
| #66 lint: update pylint configuration | craft-store | merged | Merged an update to the pylint configuration, removing 627 lines of outdated settings and adding 16 lines with no functional changes. Approved by a reviewer and passed all CI checks. | |
| #118 style: remove pylint | craft-store | merged | Merged to remove pylint from the project. Approved by one reviewer, passed CI checks, and modified two files with a net reduction of three lines. |