Merge pull request #41 from atom-ide-community/cache_templates

Author: aminya

script/vsts/platforms/cache.yml

@@ -0,0 +1,30 @@
+parameters:
+  - name: OS
+    displayName: Operating System
+    type: string
+    values:
+      - windows
+      - linux
+      - macos
+
+steps:
+  - task: Cache@2
+    displayName: Cache node_modules
+    inputs:
+      key: 'npm | "$(Agent.OS)" | "$(buildArch)" | package.json, package-lock.json, script/vsts/platforms/${{ parameters.OS }}.yml'
+      path: 'node_modules'
+      cacheHitVar: MainNodeModulesRestored
+
+  - task: Cache@2
+    displayName: Cache script/node_modules
+    inputs:
+      key: 'npm | "$(Agent.OS)" | "$(buildArch)" | script/package.json, script/package-lock.json, script/vsts/platforms/${{ parameters.OS }}.yml'
+      path: 'script/node_modules'
+      cacheHitVar: ScriptNodeModulesRestored
+
+  - task: Cache@2
+    displayName: Cache apm/node_modules
+    inputs:
+      key: 'npm | "$(Agent.OS)" | "$(buildArch)" | apm/package.json, apm/package-lock.json, script/vsts/platforms/${{ parameters.OS }}.yml'
+      path: 'apm/node_modules'
+      cacheHitVar: ApmNodeModulesRestored

script/vsts/platforms/linux.yml

@@ -30,26 +30,9 @@ jobs:
       - script: npm install --global [email protected]
         displayName: Update npm
 
-      - task: Cache@2
-        displayName: Cache node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | package.json, package-lock.json, script/vsts/platforms/linux.yml'
-          path: 'node_modules'
-          cacheHitVar: MainNodeModulesRestored
-
-      - task: Cache@2
-        displayName: Cache script/node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | script/package.json, script/package-lock.json, script/vsts/platforms/linux.yml'
-          path: 'script/node_modules'
-          cacheHitVar: ScriptNodeModulesRestored
-
-      - task: Cache@2
-        displayName: Cache apm/node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | apm/package.json, apm/package-lock.json, script/vsts/platforms/linux.yml'
-          path: 'apm/node_modules'
-          cacheHitVar: ApmNodeModulesRestored
+      - template: cache.yml
+        parameters:
+          OS: linux
 
       - script: script/bootstrap
         displayName: Bootstrap build environment

script/vsts/platforms/macos.yml

@@ -20,26 +20,9 @@ jobs:
       - script: npm install --global [email protected]
         displayName: Update npm
 
-      - task: Cache@2
-        displayName: Cache node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | package.json, package-lock.json, script/vsts/platforms/macos.yml'
-          path: 'node_modules'
-          cacheHitVar: MainNodeModulesRestored
-
-      - task: Cache@2
-        displayName: Cache script/node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | script/package.json, script/package-lock.json, script/vsts/platforms/macos.yml'
-          path: 'script/node_modules'
-          cacheHitVar: ScriptNodeModulesRestored
-
-      - task: Cache@2
-        displayName: Cache apm/node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | apm/package.json, apm/package-lock.json, script/vsts/platforms/macos.yml'
-          path: 'apm/node_modules'
-          cacheHitVar: ApmNodeModulesRestored
+      - template: cache.yml
+        parameters:
+          OS: macos
 
       - script: script/bootstrap
         displayName: Bootstrap build environment
@@ -128,26 +111,9 @@ jobs:
       - script: npm install --global [email protected]
         displayName: Update npm
 
-      - task: Cache@2
-        displayName: Cache node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | package.json, package-lock.json, script/vsts/platforms/macos.yml'
-          path: 'node_modules'
-          cacheHitVar: MainNodeModulesRestored
-
-      - task: Cache@2
-        displayName: Cache script/node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | script/package.json, script/package-lock.json, script/vsts/platforms/macos.yml'
-          path: 'script/node_modules'
-          cacheHitVar: ScriptNodeModulesRestored
-
-      - task: Cache@2
-        displayName: Cache apm/node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | apm/package.json, apm/package-lock.json, script/vsts/platforms/macos.yml'
-          path: 'apm/node_modules'
-          cacheHitVar: ApmNodeModulesRestored
+      - template: cache.yml
+        parameters:
+          OS: macos
 
       # The artifact caching task does not work on forks, so we need to
       # bootstrap again for pull requests coming from forked repositories.

script/vsts/platforms/windows.yml

@@ -48,26 +48,9 @@ jobs:
           npm install
         displayName: Install Windows build dependencies
 
-      - task: Cache@2
-        displayName: Cache node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | "$(buildArch)" | package.json, package-lock.json, script/vsts/platforms/windows.yml'
-          path: 'node_modules'
-          cacheHitVar: MainNodeModulesRestored
-
-      - task: Cache@2
-        displayName: Cache script/node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | "$(buildArch)" | script/package.json, script/package-lock.json, script/vsts/platforms/windows.yml'
-          path: 'script/node_modules'
-          cacheHitVar: ScriptNodeModulesRestored
-
-      - task: Cache@2
-        displayName: Cache apm/node_modules
-        inputs:
-          key: 'npm | "$(Agent.OS)" | "$(buildArch)" | apm/package.json, apm/package-lock.json, script/vsts/platforms/windows.yml'
-          path: 'apm/node_modules'
-          cacheHitVar: ApmNodeModulesRestored
+      - template: cache.yml
+        parameters:
+          OS: windows
 
       - script: |
           node script\vsts\windows-run.js script\bootstrap.cmd