From 478ce9e1947a70dc5b2b9365cf06dd9495567c80 Mon Sep 17 00:00:00 2001 From: Philipp Spiess Date: Wed, 9 Apr 2025 15:22:00 +0200 Subject: [PATCH] Test building a release --- .github/workflows/release-insiders.yml | 49 +++++++++++++++----------- crates/node/package.json | 5 +-- 2 files changed, 32 insertions(+), 22 deletions(-) diff --git a/.github/workflows/release-insiders.yml b/.github/workflows/release-insiders.yml index 5ad182607749..c8cbd242f2be 100644 --- a/.github/workflows/release-insiders.yml +++ b/.github/workflows/release-insiders.yml @@ -3,6 +3,7 @@ name: Release Insiders on: push: branches: [main] + pull_request: permissions: contents: read @@ -67,7 +68,7 @@ jobs: container: image: ghcr.io/napi-rs/napi-rs/nodejs-rust:lts-alpine - name: Build ${{ matrix.target }} (OXIDE) + name: Build ${{ matrix.target }} (oxide) runs-on: ${{ matrix.os }} container: ${{ matrix.container }} timeout-minutes: 15 @@ -121,9 +122,8 @@ jobs: run: pnpm install --ignore-scripts --filter=!./playgrounds/* - name: Build release - run: pnpm run --filter ${{ env.OXIDE_LOCATION }} build + run: pnpm run --filter ${{ env.OXIDE_LOCATION }} build:platform -- --target=${{ matrix.target }} env: - RUST_TARGET: ${{ matrix.target }} JEMALLOC_SYS_WITH_LG_PAGE: ${{ matrix.page-size }} - name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034 @@ -160,7 +160,7 @@ jobs: run: | sudo pkg install -y -f curl node libnghttp2 npm sudo npm install -g pnpm@9.6.0 --unsafe-perm=true - curl -sSf https://static.rust-lang.org/rustup/archive/1.27.1/x86_64-unknown-freebsd/rustup-init --output rustup-init + curl -sSf https://static.rust-lang.org/rustup/archive/1.27.1/${{ env.RUST_TARGET }} /rustup-init --output rustup-init chmod +x rustup-init ./rustup-init -y --profile minimal source "$HOME/.cargo/env" @@ -171,7 +171,7 @@ jobs: echo "~~~~ pnpm --version ~~~~" pnpm --version pnpm install --ignore-scripts --filter=!./playgrounds/* || true - pnpm run --filter ${{ env.OXIDE_LOCATION }} build + pnpm run --filter ${{ env.OXIDE_LOCATION }} build:platform -- --target=${{ env.RUST_TARGET }} strip -x ${{ env.OXIDE_LOCATION }}/*.node ls -la ${{ env.OXIDE_LOCATION }} - name: Upload artifacts @@ -213,6 +213,9 @@ jobs: cache: 'pnpm' registry-url: 'https://registry.npmjs.org' + - name: Setup WASM target + run: rustup target add wasm32-wasip1-threads + # Cargo already skips downloading dependencies if they already exist - name: Cache cargo uses: actions/cache@v4 @@ -272,19 +275,25 @@ jobs: - name: Lock pre-release versions run: node ./scripts/lock-pre-release-versions.mjs - - name: Publish - run: pnpm --recursive publish --tag ${{ env.RELEASE_CHANNEL }} --no-git-checks - env: - NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} - - - name: Trigger Tailwind Play update - uses: actions/github-script@v7 + - name: Upload artifacts + uses: actions/upload-artifact@v4 with: - github-token: ${{ secrets.TAILWIND_PLAY_TOKEN }} - script: | - await github.rest.actions.createWorkflowDispatch({ - owner: 'tailwindlabs', - repo: 'play.tailwindcss.com', - ref: 'master', - workflow_id: 'upgrade-tailwindcss.yml' - }) + name: tarballs + path: dist/*.tgz + + # - name: Publish + # run: pnpm --recursive publish --tag ${{ env.RELEASE_CHANNEL }} --no-git-checks + # env: + # NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} + + # - name: Trigger Tailwind Play update + # uses: actions/github-script@v7 + # with: + # github-token: ${{ secrets.TAILWIND_PLAY_TOKEN }} + # script: | + # await github.rest.actions.createWorkflowDispatch({ + # owner: 'tailwindlabs', + # repo: 'play.tailwindcss.com', + # ref: 'master', + # workflow_id: 'upgrade-tailwindcss.yml' + # }) diff --git a/crates/node/package.json b/crates/node/package.json index 68ba35183abb..2d0445c1fe0c 100644 --- a/crates/node/package.json +++ b/crates/node/package.json @@ -51,9 +51,10 @@ "scripts": { "artifacts": "napi artifacts", "build": "pnpm run build:platform && pnpm run build:wasm", - "postbuild": "node ./scripts/move-artifacts.mjs", "build:platform": "napi build --platform --release --no-const-enum", - "build:wasm": "napi build --release --target wasm32-wasip1-threads", + "postbuild:platform": "node ./scripts/move-artifacts.mjs", + "build:wasm": "napi build --release --target wasm32-wasip1-threads --no-const-enum", + "postbuild:wasm": "node ./scripts/move-artifacts.mjs", "dev": "cargo watch --quiet --shell 'npm run build'", "build:debug": "napi build --platform --no-const-enum", "version": "napi version"