Ensuring every build is deterministic by adding npm ci as a standard (#…

…10789) Ensuring every build is deterministic by adding npm ci as a standard option
microsoft · Aug 13, 2019 · 5d72a51 · 5d72a51
1 parent 0a63e0e
commit 5d72a51
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/Tasks/NpmV1/constants.ts b/Tasks/NpmV1/constants.ts
@@ -1,5 +1,6 @@
 export class NpmCommand {
     public static Install: string = 'install';
+    public static ContinuousIntegration: string = 'ci';
     public static Publish: string = 'publish';
     public static Custom: string = 'custom';
 }

diff --git a/Tasks/NpmV1/npm.ts b/Tasks/NpmV1/npm.ts
@@ -37,6 +37,8 @@ async function main(): Promise<void> {
     switch (command) {
         case NpmCommand.Install:
             return npmCustom.run(packagingLocation, NpmCommand.Install);
+        case NpmCommand.ContinuousIntegration:
+            return npmCustom.run(packagingLocation, NpmCommand.ContinuousIntegration);
         case NpmCommand.Publish:
             return npmPublish.run(packagingLocation);
         case NpmCommand.Custom:

diff --git a/Tasks/NpmV1/task.json b/Tasks/NpmV1/task.json
@@ -23,10 +23,10 @@
     "instanceNameFormat": "npm $(command)",
     "groups": [
         {
-            "displayName": "Custom registries and authentication",
-            "name": "customRegistries",
-            "isExpanded": false,
-            "visibleRule": "command = install || command = custom"
+          "displayName": "Custom registries and authentication",
+          "name": "customRegistries",
+          "isExpanded": false,
+          "visibleRule": "command = install || command = ci || command = custom"
         },
         {
             "name": "publishRegistries",
@@ -38,7 +38,7 @@
             "displayName": "Advanced",
             "name": "advanced",
             "isExpanded": false,
-            "visibleRule": "command = install || command = publish"
+            "visibleRule": "command = install || command = ci || command = publish"
         }
     ],
     "inputs": [
@@ -49,6 +49,7 @@
             "type": "pickList",
             "required": true,
             "options": {
+                "ci": "ci",
                 "install": "install",
                 "publish": "publish",
                 "custom": "custom"