Run release only after testing is ok also fix warning

Signed-off-by: Javier Guzman <[email protected]>
Dygmalab · Jun 23, 2023 · bd2a880 · bd2a880
1 parent becc385
commit bd2a880
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -8,6 +8,12 @@ on:
   pull_request:
     branches:
       - "main"
+  workflow_run:
+    workflows:
+      - "Testing"
+    types:
+      - completed
+
 permissions:
   contents: read
 
@@ -23,7 +29,7 @@ jobs:
       fail-fast: false
       matrix:
         include:
-          - os: macos-13
+          - os: macos-12
           - os: windows-2019
           - os: ubuntu-20.04
 

diff --git a/.github/workflows/testing.yml b/.github/workflows/testing.yml
@@ -18,7 +18,7 @@ jobs:
       fail-fast: false
       matrix:
         include:
-          - os: macos-13
+          - os: macos-12
           - os: windows-2019
           - os: ubuntu-20.04
 
@@ -42,3 +42,6 @@ jobs:
       - run: yarn run test
       # - run: yarn run lint
       # - run: yarn run test
+
+    outputs:
+      success: ${{ success() }}
diff --git a/forge.config.ts b/forge.config.ts
@@ -98,7 +98,7 @@ const config: ForgeConfig = {
       const matchString = new RegExp(`prebuilds/${platform}`);
       prebuilds.forEach(function (path) {
         if (!path.match(matchString)) {
-          fs.rmdirSync(path, { recursive: true });
+          fs.rmSync(path, { recursive: true });
         }
       });
     },