Merge pull request #160 from authzed/version-at-release-time

Set package version at release time

Author: tstirrat15

.github/workflows/publish.yaml

@@ -0,0 +1,50 @@
+---
+# yamllint disable rule:line-length
+name: "Publish to NPM"
+on:  # yamllint disable-line rule:truthy
+  release:
+    types:
+      - "published"
+jobs:
+  publish:
+    name: Publish to NPM
+    needs:
+      - test
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 18
+      - uses: bahmutov/npm-install@v1
+      - run: yarn test
+      - uses: battila7/get-version-action@v2
+      # Set the version in package.json to match the tag
+      - run: "npm version ${{ steps.get_version.outputs.version-without-v }}"
+      - uses: JS-DevTools/npm-publish@v3
+        with:
+          token: ${{ secrets.NPM_TOKEN }}
+          tag: ${{ steps.get_version.outputs.version }}
+          access: public
+  # NOTE: this publishes an alternate version that doesn't depend on protobuf-ts/runtime
+  publish-js-client:
+    name: Publish JS client to NPM
+    needs: build-js-client
+    runs-on: ubuntu-latest
+    steps:
+      - name: Download js client build
+        uses: actions/download-artifact@v4
+        with:
+          name: js-client-18
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 18
+      - uses: bahmutov/npm-install@v1
+      - uses: battila7/get-version-action@v2
+      # Set the version in package.json to match the tag
+      - run: "npm version ${{ steps.get_version.outputs.version-without-v }}"
+      - uses: JS-DevTools/npm-publish@v3
+        with:
+          token: ${{ secrets.NPM_TOKEN }}
+          tag: ${{ steps.get_version.outputs.version }}
+          access: public

.github/workflows/authzed-node.yaml .github/workflows/test.yaml.github/workflows/authzed-node.yaml renamed to .github/workflows/test.yaml

@@ -1,4 +1,4 @@
-name: authzed-node-ci
+name: "Lint and Test"
 on:
   push:
     branches:
@@ -9,8 +9,6 @@ on:
   merge_group:
     types:
       - "checks_requested"
-  release:
-    types: [published]
 jobs:
   paths-filter:
     runs-on: "buildjet-2vcpu-ubuntu-2204"
@@ -45,38 +43,12 @@ jobs:
       - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
+          cache: "yarn"
       - uses: bahmutov/npm-install@v1
-        with:
-          install-command: yarn --frozen-lockfile --silent
-          working-directory: ./
       - name: Run lint
         run: CI=true yarn lint
-        working-directory: ./
       - name: Run Yarn tests
         run: CI=true yarn only-run-tests
-        working-directory: ./
-  publish-npm:
-    name: Publish to NPM
-    needs:
-      - test
-    runs-on: ubuntu-latest
-    if: github.event_name == 'release'
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v4
-        with:
-          node-version: 18
-      - uses: bahmutov/npm-install@v1
-        with:
-          install-command: yarn --frozen-lockfile --silent
-          working-directory: ./
-      - run: yarn test
-      - uses: battila7/get-version-action@v2
-      - uses: JS-DevTools/npm-publish@v3
-        with:
-          token: ${{ secrets.NPM_TOKEN }}
-          tag: ${{ steps.get_version.outputs.version }}
-          access: public
   build-js-client:
     name: Build and Test JS client
     runs-on: "buildjet-2vcpu-ubuntu-2204"
@@ -94,16 +66,13 @@ jobs:
       - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
+          cache: "yarn"
       - uses: bahmutov/npm-install@v1
-        with:
-          install-command: yarn --frozen-lockfile --silent
-          working-directory: ./
       - name: Run build
         run: yarn build-js-client
         working-directory: ./
       - uses: bahmutov/npm-install@v1
         with:
-          install-command: yarn --frozen-lockfile --silent
           working-directory: ./js-dist
       - name: Run tests
         run: CI=true yarn only-run-tests
@@ -114,26 +83,3 @@ jobs:
           path: |
             js-dist/**
             !js-dist/node_modules/**
-  publish-js-client-npm:
-    name: Publish JS client to NPM
-    needs: build-js-client
-    runs-on: ubuntu-latest
-    if: github.event_name == 'release'
-    steps:
-      - name: Download js client build
-        uses: actions/download-artifact@v4
-        with:
-          name: js-client-18
-      - uses: actions/setup-node@v4
-        with:
-          node-version: 18
-      - uses: bahmutov/npm-install@v1
-        with:
-          install-command: yarn --frozen-lockfile --silent
-          working-directory: ./
-      - uses: battila7/get-version-action@v2
-      - uses: JS-DevTools/npm-publish@v3
-        with:
-          token: ${{ secrets.NPM_TOKEN }}
-          tag: ${{ steps.get_version.outputs.version }}
-          access: public