← Back to issue list

chore(ci): enable shellcheck

View original Github issue

Metadata

Project
charmcraft
Number
#1291
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged. The change adds a single line to enable shellcheck in the CI pipeline. Approved by two reviewers with no comments or CI failures.

Suggested action:

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. The change adds a single line to enable shellcheck in the CI pipeline. Approved by two reviewers with no comments or CI failures.
qwen3.6-35b-a3b-mtp-q6 Merged to enable shellcheck in the CI pipeline, enforcing automated shell script linting and improving build reliability.
qwen3.6-35b-a3b-mtp-q6 Enabled shellcheck in the CI workflow. Merged by maintainer lengau.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#14 chore(ci): add shellcheck and yamllint snapcraft-rocks merged 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.
83%
#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.
80%
#359 chore: delete empty shellcheckrc file starbase merged Merged deletion of an empty shellcheckrc file. The file provided no functionality and was already being removed by repository adopters. Approved by a reviewer and passed CI checks.
74%
#700 chore(docs): increment linkcheck retries rockcraft merged Merged. Increased linkcheck retries to three to prevent spurious documentation pipeline failures. Approved by three reviewers and passed CI checks.
74%
#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.
74%
#369 chore: fix linting imagecraft merged Merged after resolving redundant casting linting errors flagged by Ty. Approved by two reviewers. Changes involved minimal edits (+2/-2 lines) across one file. CI lint and test checks passed prior to merge.
72%
#345 chore: hush some of the linter output starbase merged Merged to reduce noisy linter output that flooded terminal logs. Approved by three reviewers, passed CI, and applied a two-line change to silence specific warnings.
72%
#1123 chore: remove comment fences from GitHub templates rockcraft merged Merged a chore PR removing HTML comment fences from GitHub templates to improve contributor readability. Approved by two reviewers and passed CI checks.
72%
#5561 chore: fix dotnet comment snapcraft merged Merged a maintainer PR updating an outdated dotnet plugin comment. Approved by two reviewers, passed CI checks, and changed one file (+2/-1).
72%
#271 chore(ci): generate release notes when publishing starbase merged Merged a maintainer-submitted change that automates release note generation during publishing via CI. Approved by two reviewers, passed all checks, and added two lines to a single file.
72%