ci: Fix CI filter job and linting issues
Metadata
Current evaluation
Merged in PR #2740, adding checkout to the policy workflow filter job and removing an unused ty: ignore, fixing the git and lint CI failures.
Resolution: closed resolved
Reason: PR #2740 was merged with approval from two reviewers and all CI checks passing. It restored the missing checkout step and corrected the lint failure.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This PR fixes two issues:
1. Added missing checkout step to policy.yaml workflow to resolve git error in filter job
2. Removed unused ty: ignore comment that was causing linting failure
---
- [ ] I've followed the [contribution guidelines](https://github.com/canonical/charmcraft/blob/main/CONTRIBUTING.md).
- [ ] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [ ] I've successfully run `make lint && make test`.
- [ ] I've added or updated any relevant documentation.
- [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing.
- [ ] I've updated the relevant release notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in PR #2740, adding checkout to the policy workflow filter job and removing an unused ty: ignore, fixing the git and lint CI failures. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged CI fix added checkout to the policy filter workflow and removed an unused ty: ignore, resolving both reported failures. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2740 added a missing checkout step to policy.yaml, fixing the filter job's git error, and removed an unused ty: ignore comment to fix linting. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged changes resolving CI filter job git errors and linting failures. Added a missing checkout step to policy.yaml and removed an unused ty: ignore comment. Approved by two reviewers with all CI checks passing. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer lengau. Resolved a git error in the filter job by adding a missing checkout step to policy.yaml and fixed linting failures by removing an unused ty: ignore comment. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #567 ci: fix linter issues | craft-parts | merged | PR #567 merged as c41f86d6, fixing CI linter issues across four files after approval. | |
| #1191 ci: Fix QA workflow permissions for reusable lint job | craft-application | closed | PR #1191 was approved but closed unmerged; commit 5353ad50 records its lint permission change, while lint/files and title-policy checks still failed. | |
| #1204 style: fix linting issues | craft-parts | closed | Lint-only cleanup landed in commit a0622b31, matching PR #1204’s title; all reported CI checks passed. |