← Back to issue list

feat: add linting CI workflow

View original Github issue

Metadata

Project
craft-actions
Number
#33
Type
pull request
State
closed
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

PR #33 was closed unmerged after 28 days without reviews, comments, or CI checks. Linting was later added by merged PR #63 (c2cf7cb).

Resolution: closed stale

Reason: PR #33 received no reviews, comments, or CI checks and was closed unmerged after 28 days of inactivity. A later merged PR #63, commit c2cf7cb, introduced the current ESLint configuration and pnpm lint CI step.

Evaluated by: mr-cal-bot

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed stale PR #33 was closed unmerged after 28 days without reviews, comments, or CI checks. Linting was later added by merged PR #63 (c2cf7cb).
stealth/space-bunny-alpha — closed stale Lint CI workflow PR #33 was closed unmerged after 28 days of inactivity, with no reviews, comments or CI checks.
stealth/space-bunny-alpha — closed stale Lint CI workflow PR #33 was closed unmerged after 28 days of inactivity, with no reviews, comments, or CI checks.
stealth/space-bunny-alpha — closed stale Lint CI workflow PR #33 was closed unmerged after 28 days of inactivity, without reviews, comments, or CI checks.
qwen/qwen3.6-35b-a3b — — Changes to add a linting CI workflow were closed without merging. No reviews or comments were submitted, and the branch was abandoned after 28 days of inactivity.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/craft-actions#63 (confidence 95%)

    Merged PR #63 added pnpm-based lint tooling, ESLint configuration, and a lint step to the test workflow.

Related issues

Issue Project State Summary Similarity
#32 [internal] Add linting workflow craft-actions closed Lint workflow request was closed without a documented implementation, resolving PR, commit, or linked follow-up.
76%
#33 Linting imagecraft merged Merged the one-line linting cleanup from PR #33 as commit aed3b2d via merge commit 44d5d39.
74%
#1322 style: resolve linting issues rockcraft closed PR #1322 was closed unmerged after 42 days of inactivity; lint checks passed, but source-spread-tests failed and 10 review comments remained unresolved.
73%
#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.
72%
#567 ci: fix linter issues craft-parts merged PR #567 merged as c41f86d6, fixing CI linter issues across four files after approval.
72%
#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.
71%
#20 style: fix linting issues craft-artifacts merged Merged PR #20 via commit 04cbdc0, applying style-only lint fixes across four files (+11/-5); both reviews approved and all CI checks passed.
71%
#77 lint: fix linting issues with up-to-date linters craft-store merged Merged PR #77 fixed lint failures with current linters across 11 files (+40/-24), with one approval and no unresolved review comments.
71%
#15 feat: make the lint workflow reusable starflow closed Closed unmerged; the reusable lint workflow landed in #18 via commit cd64717, while downstream craft-platforms#60 adopted Starflow workflows.
70%
#216 ci: Add missing permissions to QA lint workflow craft-grammar closed PR #216 (lengau) added explicit contents/pull-requests read permissions to the QA lint job and was approved, but the lint / files check failed; the PR was closed unmerged after ~8 days with no resolution recorded.
70%