Skip to content

Commit c52caf3

Browse files
tbjersclaude
andcommitted
ci: reference the report action by local path, not a version pin
Point both Report coverage steps at ./.github/actions/report instead of a pinned CoverageTracker/coverage-tracker/.../report@vX.Y.Z tag. This repo dogfoods its own action, so it should run the exact code on the current branch rather than a released version — and a local path never needs resolving a tag/branch at all, which permanently removes the catch-22 where release-prep bumped this pin ahead of the tag existing. Reverts the now-unnecessary release-prepare.yml changes (ci.yml pin bumping, workflows: write) since there's no longer a pin here to bump. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent 679e225 commit c52caf3

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

.github/workflows/ci.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,11 @@ jobs:
7979
retention-days: 30
8080

8181
- name: Report coverage
82-
uses: CoverageTracker/coverage-tracker/.github/actions/report@v0.4.0
82+
# Local path, not a pinned release tag: this repo dogfoods its own
83+
# action, so it should always run the exact code on this branch/ref
84+
# rather than a released version that may lag behind (or, during
85+
# release prep, not exist yet as a tag at all).
86+
uses: ./.github/actions/report
8387
with:
8488
worker-url: https://demo.coveragetracker.dev
8589
coverage-path: dashboard/coverage/lcov.info
@@ -115,7 +119,7 @@ jobs:
115119
run: npm run test:coverage
116120

117121
- name: Report coverage
118-
uses: CoverageTracker/coverage-tracker/.github/actions/report@v0.4.0
122+
uses: ./.github/actions/report
119123
with:
120124
worker-url: https://demo.coveragetracker.dev
121125
category: backend

.github/workflows/release-prepare.yml

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,6 @@ on:
1717
permissions:
1818
contents: write
1919
pull-requests: write
20-
# Needed because the release commit touches .github/workflows/ci.yml
21-
# (bumping its action-usage pin) — the GitHub API rejects workflow-file
22-
# writes from a token without this permission, even with contents: write.
23-
workflows: write
2420

2521
concurrency:
2622
group: release
@@ -82,10 +78,10 @@ jobs:
8278
(cd "$dir" && npm version "$VERSION" --no-git-tag-version --allow-same-version)
8379
done
8480
85-
- name: Update action-usage pins
81+
- name: Update README action-usage pins
8682
run: |
8783
sed -i -E "s#(\.github/actions/report@v)[0-9]+\.[0-9]+\.[0-9]+#\1${VERSION}#g" \
88-
README.md .github/actions/report/README.md .github/workflows/ci.yml
84+
README.md .github/actions/report/README.md
8985
9086
- name: Create release branch
9187
env:
@@ -104,7 +100,7 @@ jobs:
104100
package.json package-lock.json \
105101
dashboard/package.json dashboard/package-lock.json \
106102
.github/actions/report/package.json .github/actions/report/package-lock.json \
107-
CHANGELOG.md README.md .github/actions/report/README.md .github/workflows/ci.yml
103+
CHANGELOG.md README.md .github/actions/report/README.md
108104
109105
- name: Open release PR
110106
env:

0 commit comments

Comments
 (0)