Skip to content

Commit 5452ac5

Browse files
authored
ci: Build test-snaps in publish-release job to include it in NPM publishing (#3712)
Following #3679, we want `test-snaps` to be publishable to NPM. Currently only TypeScript packages are built, so `test-snaps` wouldn't be published properly. I've added an extra step to build `test-snaps` before running the NPM publish job. <!-- CURSOR_SUMMARY --> --- > [!NOTE] > Updates the publish-release workflow to install dependencies in a dedicated step and build `@metamask/test-snaps` so it’s included in release artifacts/NPM publish. > > - **CI (GitHub Actions)**: > - **`publish-release` job**: > - Add dedicated `Install dependencies` step (`yarn install --immutable`). > - Build all packages with `yarn build`. > - New step to build `@metamask/test-snaps` (`yarn workspace @metamask/test-snaps build`) so artifacts include it. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 4ffbc25. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
1 parent 313198b commit 5452ac5

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

.github/workflows/publish-release.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,10 +62,12 @@ jobs:
6262
npm-tag: ${{ needs.get-release-tag.outputs.tag }}
6363
env:
6464
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
65+
- name: Install dependencies
66+
run: yarn install --immutable
6567
- name: Build
66-
run: |
67-
yarn install --immutable
68-
yarn build
68+
run: yarn build
69+
- name: Build test-snaps
70+
run: yarn workspace @metamask/test-snaps build
6971
- name: Upload build artifacts
7072
uses: actions/upload-artifact@v4
7173
with:

0 commit comments

Comments
 (0)