← Back to issue list

lint: run shellcheck on spread.yaml

View original Github issue

Metadata

Project
rockcraft
Number
#201
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #201 in commit c90598eb, enabling ShellCheck linting for spread.yaml through a small lint-configuration update.

Resolution: closed resolved

Reason: PR #201 was approved and merged. Commit c90598eb applies the configuration change that runs ShellCheck on spread.yaml.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #201 in commit c90598eb, enabling ShellCheck linting for spread.yaml through a small lint-configuration update.
stealth/space-bunny-alpha — closed resolved Merged PR #201 as commit c90598eb, enabling ShellCheck on spread.yaml and updating the lint configuration.
stealth/space-bunny-alpha — closed resolved Merged PR #201 (commit c90598eb) enabled shellcheck for spread.yaml through a small lint-configuration change.
qwen/qwen3.6-35b-a3b — — Pull request to run shellcheck on spread.yaml was approved, passed CI, and merged with a four-line change across two files.
qwen3.6-35b-a3b-mtp-q6 — — Merged to add shellcheck linting for spread.yaml, enhancing shell script validation in the project's configuration and testing workflows.

Update history

No update history recorded yet.

Related work

  • Related To: rockcraft#196 (confidence 90%)

    PR #196 previously added the ShellCheck and spread-shellcheck linters used by this configuration update.

Related issues

Issue Project State Summary Similarity
#196 lint: add shellcheck and spread-shellcheck linters rockcraft merged Merged PR #196 as b7c4c794, adding ShellCheck and spread-shellcheck to GitHub CI plus minor compliance fixes.
83%
#9 lint spread tests with shellcheck snapcraft-rocks closed Added ShellCheck linting for spread tests via merged PR canonical/snapcraft-rocks#14 ("chore(ci): add shellcheck and yamllint"), which also enabled yamllint. Maintainer tigarmo closed the issue with "Fixed by #14".
83%
#2381 tools: copy in spread-shellcheck from snapd snapcraft merged Merged PR #2381 as cce7842e8, adding snapd's spread-shellcheck tool, expanding ShellCheck coverage to spread tests, and correcting affected glob patterns.
74%
#291 lint: add shellcheck, yamllint craft-providers merged Merged PR #291 in commit 68ac1cc, adding shellcheck and yamllint from Starbase to address #287.
71%