-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
GitOrigin-RevId: eb577bdd9e3ba97c89cd7601c90acd62cc747d88
- Loading branch information
1 parent
393a77a
commit 6c4158f
Showing
2 changed files
with
9 additions
and
25 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,9 +23,14 @@ jobs: | |
version: latest | ||
- name: Run Biome | ||
run: biome ci . | ||
validate: | ||
release: | ||
needs: [quality] | ||
runs-on: ubuntu-latest | ||
permissions: | ||
contents: write # to be able to publish a GitHub release | ||
issues: write # to be able to comment on released issues | ||
pull-requests: write # to be able to comment on released pull requests | ||
id-token: write # to enable use of OIDC for npm provenance | ||
steps: | ||
- name: Check out code | ||
uses: actions/checkout@v4 | ||
|
@@ -36,7 +41,7 @@ jobs: | |
with: | ||
bun-version: latest | ||
- name: Install dependencies (`npm ci`) | ||
# Different set of dependencies due to subset of packages causes lockfile changes that cause failure, so we don't copy lockfile over | ||
# Different set of dependencies due to subset of packages causes lockfile changes that cause failure so no --frozen-lockfile | ||
# https://github.com/oven-sh/bun/issues/5792#issuecomment-2325444077 | ||
run: bun install | ||
- name: Test | ||
|
@@ -46,27 +51,6 @@ jobs: | |
run: bun --filter react-router-busy build | ||
- name: Check exports | ||
run: bun --filter react-router-busy check-exports | ||
release: | ||
needs: [validate] | ||
runs-on: ubuntu-latest | ||
permissions: | ||
contents: write # to be able to publish a GitHub release | ||
issues: write # to be able to comment on released issues | ||
pull-requests: write # to be able to comment on released pull requests | ||
id-token: write # to enable use of OIDC for npm provenance | ||
steps: | ||
- name: Check out code | ||
uses: actions/checkout@v4 | ||
with: | ||
fetch-depth: 0 | ||
- name: Set up Bun | ||
uses: oven-sh/[email protected] | ||
with: | ||
bun-version: latest | ||
scope: "react-router-busy" | ||
- name: Install dependencies (`npm ci`) | ||
# Different set of dependencies due to subset of packages causes lockfile changes that cause failure, so we don't copy lockfile over | ||
run: bun install | ||
# Note we have to toggle Workflow permissions > Allow GitHub Actions to create and approve pull requests in https://github.com/bitofbreeze/mono/settings/actions | ||
# Even with permissions above https://github.com/orgs/community/discussions/27689#discussioncomment-5707424 | ||
# Could avoid this with personal access token https://microsoft.github.io/beachball/concepts/ci-integration.html#authentication | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters