Skip to content

ci: pin app-builder by its FULL sha — a 7-char ref does not resolve - #10

Merged
drsamonline merged 1 commit into
mainfrom
forge/pin-full-sha-2026-11-12
Nov 12, 2026
Merged

drsamonline merged 1 commit into
mainfrom
forge/pin-full-sha-2026-11-12

Conversation

@drsamonline

Copy link
Copy Markdown
Owner

.github/workflows/build-apps.yml pins the shared pipeline with a 7-character ref:

uses: drsamonline/app-builder/.github/workflows/build.yml@49c2e0b

A reusable-workflow uses: ref must be a full commit SHA, a tag or a branch. GitHub cannot resolve the abbreviation, so the workflow is rejected at startup: no job at all, referenced_workflows: [], and the run is named by its path (.github/workflows/build-apps.yml) instead of Build + deploy.

Evidence

  • This repo: last green Build + deploy before 2026-11-11 14:20Z, red from 14:20Z onward, with no job to read.
  • The same file with a full-length ref was green: retro-strike went success 2026-11-10T15:34Z → failure 2026-11-11T14:24Z, and the only change in that commit is the ref.
  • actionlint passes the file, and the called workflow declares every input and permission the caller passes — so it is the ref, not the YAML.
  • The commit exists: gh api /repos/drsamonline/app-builder/commits/49c2e0b → 49c2e0b172890f9334bbc7167a0e7fb5cd3756ba.
  • Canary, merged-run proof: app-panchang#6 with this exact change ran Build + deploy → success.

The change

The same commit, written out in full. One line, nothing else.

Part of a 37-repo sweep from the same cause (all of them the identical @49c2e0b).

A reusable-workflow `uses:` ref must be a full commit SHA, a tag or a
branch. @49c2e0b is 7 characters, so GitHub cannot resolve it and the
workflow is rejected at STARTUP with no job at all: the run is named by
its path and referenced_workflows is empty.

Measured: the same file with a full-length ref was green (retro-strike
2026-11-10T15:34Z); this repo's only change since is the ref. The canary
with this exact change (app-panchang#6) built green.
@drsamonline
drsamonline merged commit cbca27e into main Nov 12, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant