Run the quality checks from a script file #90

Merged
Tirsvad merged 3 commits from quality-checks-script-file into main 2026-10-11 05:15:53 +02:00
Owner

The shell of the reusable quality-checks workflow moves to tools/quality-checks.sh; quality-checks.yml shrinks from about 150 to 74 lines and embeds no script.

  • tools/quality-checks.sh validate|install|run: the three steps unchanged in behavior and messages.
  • quality-checks.yml: checks out the caller, fetches the script into shared-scripts/ at the new input tools-ref (default v1) and runs it.
  • Why an input: a called workflow cannot learn its own ref on this instance (workflow_ref, workflow_sha, job_workflow_sha, action_ref are empty; verified on a throwaway branch). A call at an exact tag is pinned completely only with tools-ref set to the same tag; documented in the guide and the upgrade guide.
  • Tests run the script file with the same stub tools; new tests for the input, the fetch step and the unknown command. 238 pass.
  • ADR-0005 notes the input (RC-051); changelog entry for 1.1.0 (a new optional input is a minor change).
  • Trial: RepoFoundry throwaway branch calling this branch with tools-ref set to it, run 663 passed (shellcheck and shfmt); the branch is deleted.

Wiki pages changed: QualityChecksPythonShell and Upgrading (en, da, ms).

Order matters: on main the default tools-ref: v1 points to 1.0.1, which has no script yet, so a caller at @main (RepoFoundry today) fails until 1.1.0 is released right after this merge.

Refs #24

The shell of the reusable quality-checks workflow moves to `tools/quality-checks.sh`; `quality-checks.yml` shrinks from about 150 to 74 lines and embeds no script. - `tools/quality-checks.sh validate|install|run`: the three steps unchanged in behavior and messages. - `quality-checks.yml`: checks out the caller, fetches the script into `shared-scripts/` at the new input `tools-ref` (default `v1`) and runs it. - Why an input: a called workflow cannot learn its own ref on this instance (`workflow_ref`, `workflow_sha`, `job_workflow_sha`, `action_ref` are empty; verified on a throwaway branch). A call at an exact tag is pinned completely only with `tools-ref` set to the same tag; documented in the guide and the upgrade guide. - Tests run the script file with the same stub tools; new tests for the input, the fetch step and the unknown command. 238 pass. - ADR-0005 notes the input (RC-051); changelog entry for `1.1.0` (a new optional input is a minor change). - Trial: RepoFoundry throwaway branch calling this branch with `tools-ref` set to it, run 663 passed (shellcheck and shfmt); the branch is deleted. Wiki pages changed: QualityChecksPythonShell and Upgrading (en, da, ms). Order matters: on `main` the default `tools-ref: v1` points to `1.0.1`, which has no script yet, so a caller at `@main` (RepoFoundry today) fails until `1.1.0` is released right after this merge. Refs #24
Tirsvad added 3 commits 2026-10-11 05:15:17 +02:00
Task: MIL-003#3
Resolve pending commit links in the quality-checks script documents
TirSystem/github-action: Publish wiki / publish-wiki (push) Skipped
TirSystem/github-action: Sync GitHub mirror metadata / sync-metadata (push) Skipped
ae0360350a
Task: MIL-003#3
Add the changelog entry of release 1.1.0
TirSystem/github-action: Publish wiki / publish-wiki (push) Skipped
TirSystem/github-action: Sync GitHub mirror metadata / sync-metadata (push) Skipped
Validate workflows / validate (pull_request) Successful in 14s
0025c58e7b
Task: MIL-004#5
Tirsvad added 1 commit 2026-10-11 05:15:18 +02:00
Add the changelog entry of release 1.1.0
TirSystem/github-action: Publish wiki / publish-wiki (push) Skipped
TirSystem/github-action: Sync GitHub mirror metadata / sync-metadata (push) Skipped
Validate workflows / validate (pull_request) Successful in 14s
0025c58e7b
Task: MIL-004#5
Tirsvad merged commit 2817ecdef5 into main 2026-10-11 05:15:53 +02:00
Tirsvad deleted branch quality-checks-script-file 2026-10-11 05:15:53 +02:00
Sign in to join this conversation.