Skip to content

Commit ce47314

Browse files
farhanclaude
andcommitted
fix: ensure release.yml calls CI first via run_tests reusable workflow
Add a `run_tests` job that calls `.github/workflows/ci.yml` as a reusable workflow, and make `release` depend on it via `needs: run_tests`. Move permissions from workflow-level to job-level for least-privilege. No PYPI_UPLOAD_TOKEN or password fields — OIDC pattern preserved. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent e401671 commit ce47314

1 file changed

Lines changed: 10 additions & 4 deletions

File tree

.github/workflows/release.yml

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,20 @@ on:
44
tags:
55
- '*'
66

7-
permissions:
8-
id-token: write # Required for OIDC
9-
contents: write # For Semantic Release tagging
10-
117
jobs:
8+
run_tests:
9+
uses: ./.github/workflows/ci.yml
10+
secrets: inherit
11+
permissions:
12+
contents: read
13+
1214
release:
1315
name: Release
16+
needs: run_tests
1417
runs-on: ubuntu-latest
18+
permissions:
19+
id-token: write # Required for OIDC
20+
contents: write # For Semantic Release tagging
1521
steps:
1622
- name: Checkout
1723
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4.3.1

0 commit comments

Comments
 (0)