← Back to issue list

chore(ci): add shellcheck and yamllint

View original Github issue

Metadata

Project
snapcraft-rocks
Number
#14
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged. Resolves issue #9 by adding shellcheck and yamllint to the CI pipeline. Approved by one reviewer. Updates 5 files with 49 additions and 7 deletions.

Suggested action:

No scores available.

Issue body

Fixes #9

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Resolves issue #9 by adding shellcheck and yamllint to the CI pipeline. Approved by one reviewer. Updates 5 files with 49 additions and 7 deletions.
qwen3.6-35b-a3b-mtp-q6 Merged, integrating shellcheck and yamllint into the CI pipeline. Resolves issue #9. Changes were applied directly by the maintainer without additional comments.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1291 chore(ci): enable shellcheck charmcraft merged Merged. The change adds a single line to enable shellcheck in the CI pipeline. Approved by two reviewers with no comments or CI failures.
83%
#362 chore: remove unused yamllint dependency starbase merged Merged a maintainer's change to remove the unused yamllint dependency. Approved by one reviewer, the update modified three files, removing 29 lines and adding 3.
79%
#5183 chore(merge): 8.5.1 into main snapcraft merged Merged version 8.5.1 into main. Approved by two reviewers, passed CI checks, and updated six files with 92 additions and 27 deletions.
76%
#5489 chore: remove unused yamllint config snapcraft merged Merged after approval by two reviewers. The update removes an unused yamllint configuration file, deleting 29 lines across one file. The change passed most CI checks and was successfully integrated into the repository.
76%
#5287 chore: delete empty shellcheckrc file snapcraft merged Merged PR removing an accidentally committed empty shellcheckrc file. Approved by two reviewers and passed CI, successfully cleaning the repository root.
76%
#840 chore(merge): 5.6.2 into main craft-application merged Merged version 5.6.2 into main following successful CI checks and approval from two reviewers. The commit updates three files with 44 additions and 5 deletions.
76%
#375 chore(lint): Remove trailing whitespace rockcraft merged Merged changes removing trailing whitespace from one file to satisfy linters like yamllint. Approved by two reviewers, passed CI, and modified four lines.
76%
#529 chore: fix doc and yaml linting craft-parts merged Merged after two approvals and passing CI. Resolves documentation and YAML linting issues across three files. Net reduction of two lines. Local verification available via tox lint.
76%
#585 chore(linter): misc fixes craft-parts merged Merged linter fixes across 125 files. Approved by two reviewers and merged without squashing per maintainer request.
76%
#907 chore(merge): 2.1.3 into main craft-parts merged Merged version 2.1.3 into main following approval from two reviewers and passing CI checks. The change updated seven files, adding 36 lines and removing 10.
75%