Engineering Fixes: PR Validation, CI runs (#28)

* npm clean

* CI only for master

* CI only for master

* Move npm install to package.json

Author: saikrishnav

.github/workflows/build.yml

@@ -1,6 +1,14 @@
 name: Node CI
 
-on: [push, pull_request]
+on: 
+  push:
+    branches:
+      - master
+      - 'releases/**'
+  
+  pull_request: 
+    branches:
+      - '*'
 
 jobs:
   build:
@@ -18,8 +26,7 @@ jobs:
       with:
         node-version: ${{ matrix.node-version }}
     - name: npm install, build, and test
-      run: |
-        npm install --no-optional
+      run: |        
         npm run build --if-present
       env:
         CI: true

Tasks/emailReportTask/config/ConfigurationProvider.ts

@@ -166,4 +166,4 @@ export class ConfigurationProvider implements IConfigurationProvider {
       default: return GroupTestResultsBy.Run;
     }
   }
-}
+}

package.json

@@ -4,7 +4,7 @@
   "description": "",
   "scripts": {
     "test": "npm run jest",
-    "build": "npm run build:t1 && npm run build:t2",
+    "build": "npm install --no-optional && npm run build:t1 && npm run build:t2",
     "build:t1": "npm run build:emailreport",
     "build:t2": "npm run build:prinsights",
     "pack:t1": "npm run clean && npm run build:t1 && npm run pack:emailreport",
@@ -17,7 +17,7 @@
     "pack:prinsights": "node ./commands/package.js pullRequestInsights",
     "e2e:emailreport": "node js/emailReportExtension/emailReportTask/tests/__e_to_e_tests__/InvokeTest.js",
     "e2e:prinsights": "node js/pullRequestInsightsExtension/pullRequestInsightsTask/tests/__e_to_e_tests__/InvokeTest.js",
-    "clean": "rimraf dist *.vsix"
+    "clean": "rimraf dist && rimraf node_modules"
   },
   "repository": {
     "type": "git",