Merge pull request #168 from authzed/remove-lockfiles

Remove lockfiles

Author: tstirrat15

.github/workflows/automatic-api-update.yaml

@@ -40,6 +40,7 @@ jobs:
         uses: bahmutov/npm-install@v1
         if: steps.buf-update.outputs.updated == 'true'
         with:
+          useLockFile: false
           working-directory: ./
       - name: "Install buf"
         uses: "bufbuild/buf-setup-action@v1.39.0"

.github/workflows/manual-api-update.yaml

@@ -44,6 +44,7 @@ jobs:
         uses: bahmutov/npm-install@v1
         if: steps.buf-update.outputs.updated == 'true'
         with:
+          useLockFile: false
           working-directory: ./
       - name: "Install buf"
         uses: "bufbuild/buf-setup-action@v1.39.0"

.github/workflows/publish.yaml

@@ -14,8 +14,12 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
+          cache-dependency-path: ./package.json
+          cache: "yarn"
           node-version: 18
       - uses: bahmutov/npm-install@v1
+        with:
+          useLockFile: false
       # Set the version in package.json to match the tag
       - name: Write release version
         run: |
@@ -39,13 +43,17 @@ jobs:
       - uses: actions/setup-node@v4
         with:
           node-version: 18
+          cache-dependency-path: ./package.json
           cache: "yarn"
       # Install deps in base package and build into js-dist
       - uses: bahmutov/npm-install@v1
+        with:
+          useLockFile: false
       - name: Run build
         run: yarn build-js-client
       - uses: bahmutov/npm-install@v1
         with:
+          useLockFile: false
           working-directory: ./js-dist
       # Set the version in package.json to match the tag
       - name: Write release version

.github/workflows/test.yaml

@@ -40,8 +40,11 @@ jobs:
       - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
+          cache-dependency-path: ./package.json
           cache: "yarn"
       - uses: bahmutov/npm-install@v1
+        with:
+          useLockFile: false
       - name: Run lint
         run: CI=true yarn lint
       - name: Run Yarn tests
@@ -63,13 +66,17 @@ jobs:
       - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
+          cache-dependency-path: ./package.json
           cache: "yarn"
       - uses: bahmutov/npm-install@v1
+        with:
+          useLockFile: false
       - name: Run build
         run: yarn build-js-client
         working-directory: ./
       - uses: bahmutov/npm-install@v1
         with:
+          useLockFile: false
           working-directory: ./js-dist
       - name: Run tests
         run: CI=true yarn only-run-tests

.gitignore

@@ -4,3 +4,12 @@ src/authzedapi/authzed/api/v0
 src/authzedapi/authzed/api/v1alpha1
 js-dist/src/authzedapi/authzed/api/v0
 js-dist/src/authzedapi/authzed/api/v1alpha1
+
+# Because of how node's dependency management works,
+# and the fact that a lockfile in a published package
+# isn't exported, keeping these files out of the repo
+# will make our tests reflect installed usage more closely.
+yarn.lock
+package-lock.json
+js-dist/yarn.lock
+js-dist/package-lock.json

.npmrc

@@ -0,0 +1 @@
+package-lock=false