Merge pull request #17 from brown-ccv/develop

ci: update workflows
brown-ccv · Jul 20, 2020 · cb3276c · cb3276c
2 parents 85a4e24 + 7e0ce1a
commit cb3276c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -31,6 +31,8 @@ jobs:
     # install, test, build package
     - name: npm install
       run: npm install
+    - name: get git revision and branch
+      run: node version.js
     - name: npm build
       run: npm run build
 

diff --git a/.github/workflows/make-release.yml b/.github/workflows/make-release.yml
@@ -29,6 +29,8 @@ jobs:
     # install, test, build package
     - name: npm install
       run: npm install
+    - name: get git revision and branch
+      run: node version.js
     - name: npm build - linux mac
       if: startsWith(matrix.os, 'mac') || startsWith(matrix.os, 'ubuntu')
       run: npm run build
@@ -37,8 +39,6 @@ jobs:
       run: npm run build:win:clinic
     - name: npm test
       run: npm test
-      env:
-        CI: true
 
     # Build electron app package installers
     - name: package electron - windows
@@ -92,9 +92,6 @@ jobs:
         REACT_APP_AT_HOME: true
     - name: npm test - win home
       run: npm test
-      env:
-        CI: true
-        REACT_APP_AT_HOME: true
     - name: package electron - windows home
       if: startsWith(matrix.os, 'windows')
       run: npm run package:windows