Skip to content

fix(ci): update publish workflow ci-templates SHA#67

Merged
Jesssullivan merged 1 commit intomainfrom
fix/publish-ci-templates-sha
Apr 18, 2026
Merged

fix(ci): update publish workflow ci-templates SHA#67
Jesssullivan merged 1 commit intomainfrom
fix/publish-ci-templates-sha

Conversation

@Jesssullivan
Copy link
Copy Markdown
Owner

Summary

Without this fix, any tag push or release creation will fail with startup_failure before jobs are created.

Tracked

  • TIN-104 (Bazel artifact truth)
  • TIN-164 (ci-templates Bazel workflow)

The publish.yml still pinned to @0e13cc4 which has per-job permissions
blocks that cause startup_failure on GitHub Actions. CI workflow was
already updated in #66 but publish was missed.

Aligns with ci-templates PRs #10 + #11 (runs-on fix + permissions fix).

Tracked: TIN-104, TIN-164.
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 18, 2026

Greptile Summary

This PR updates the publish.yml workflow's ci-templates SHA from 0e13cc4 to f2e12c1, matching the SHA already used in ci.yml (updated in #66). The old SHA had per-job permissions blocks that caused startup_failure, preventing tag pushes and release events from working at all.

Confidence Score: 5/5

Safe to merge — single-line SHA bump that aligns publish.yml with the already-working ci.yml reference.

The change is a one-line SHA update with a clear, documented rationale. Both workflows now reference the same ci-templates commit, restoring the publish path that was broken by the missed update in #66. No logic, configuration, or secrets handling is modified.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/publish.yml SHA bump for ci-templates reusable workflow from 0e13cc4 to f2e12c1, aligning with the SHA already used in ci.yml; no other changes.

Sequence Diagram

sequenceDiagram
    participant GH as GitHub
    participant PW as publish.yml
    participant CT as ci-templates<br/>js-bazel-package.yml<br/>@f2e12c1
    participant Runner as Self-hosted Runner

    GH->>PW: release published / workflow_dispatch
    PW->>CT: uses: reusable workflow (top-level permissions)
    CT->>Runner: spawn jobs (lint, typecheck, test, build, package)
    Runner-->>CT: job results
    CT-->>PW: workflow conclusion
    PW-->>GH: publish outcome
Loading

Reviews (1): Last reviewed commit: "fix(ci): update publish workflow ci-temp..." | Re-trigger Greptile

@Jesssullivan Jesssullivan merged commit 27b992b into main Apr 18, 2026
5 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