ci: remove scripts.postinstall in CI scripts (#130)

* ci: remove scripts.postinstall in CI scripts * build: remove prepublishOnly script Signed-off-by: Rong Sen Ng (motss) <wes.ngrongsen@gmail.com>
motss · Nov 21, 2022 · d593ffb · d593ffb
1 parent fc2c01f
commit d593ffb
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 17 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -1,16 +1,11 @@
 name: ci
 
 on:
-  push:
-    branches:
-      - '*'
-    tags-ignore:
-      - '*'
   pull_request:
     branches:
-      - '*'
-    tags-ignore:
-      - '*'
+      - '**'
+    paths-ignore:
+      - '**/*.md'
 
 concurrency:
       group: ${{ github.workflow }}-${{ github.ref || github.run_id }}
@@ -37,7 +32,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest]
-        target: [14.x, 16.x, 18.x]
+        target: [16.x, 18.x]
 
     steps:
       - name: Checkout
@@ -69,6 +64,7 @@ jobs:
 
       - name: Install dependencies
         run: |
+          npm pkg delete scripts.postinstall
           npm ci
           sh $(npm root)/@reallyland/tools/list-npx-cache.sh
 
@@ -86,7 +82,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest]
-        target: [14.x, 16.x, 18.x]
+        target: [16.x, 18.x]
 
     steps:
       - name: Checkout
@@ -123,6 +119,7 @@ jobs:
 
       - name: Install dependencies
         run: |
+          npm pkg delete scripts.postinstall
           npm ci
           sh $(npm root)/@reallyland/tools/list-npx-cache.sh
 
@@ -154,7 +151,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest]
-        target: [14.x, 16.x, 18.x]
+        target: [16.x, 18.x]
 
     steps:
       - name: Checkout
@@ -186,6 +183,7 @@ jobs:
 
       - name: Install dependencies
         run: |
+          npm pkg delete scripts.postinstall
           npm ci
           sh $(npm root)/@reallyland/tools/list-npx-cache.sh
 

diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -2,10 +2,8 @@ name: publish
 
 on:
   push:
-    branches-ignore:
-      - '*'
     tags:
-      - '*'
+      - 'v**'
 
 defaults:
   run:
@@ -25,7 +23,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest]
-        target: [16.x]
+        target: [18.x]
 
     steps:
       - name: Checkout
@@ -63,8 +61,21 @@ jobs:
 
       - name: Install dependencies
         run: |
+          npm pkg delete scripts.postinstall
           npm ci
 
+      - name: Lint
+        run: |
+          npm run lint:build
+
+      - name: Build
+        run: |
+          npm run build
+
+      - name: Cleanup package.json
+        run: |
+          npm pkg delete devDependencies nano-staged scripts
+
       - name: Publish to npm (Prerelease)
         if: contains(github.ref, '-rc') == true || contains(github.ref, '-') == true
         run: |

diff --git a/package.json b/package.json
@@ -67,9 +67,7 @@
     "lint-commit": "sh $(npm root)/@reallyland/tools/lint-commit.sh",
     "lint:build": "sh $(npm root)/@reallyland/tools/lint-build.sh -c $(npm root)/@reallyland/tools/browser/.build.eslintrc.json",
     "pre-commit": "sh $(npm root)/@reallyland/tools/pre-commit.sh",
-    "prepublishOnly": "npm run lint:build && npm run build",
     "test": "vitest --coverage",
-    "ts": "tsc",
     "version": "sh $(npm root)/@reallyland/tools/generate-changelogs.sh && git add *CHANGELOG.md"
   },
   "dependencies": {