screwdriver-executor-router - npm Package Compare versions

Comparing version 0.1.1 to 0.1.2

index.js

@@ -20,2 +20,3 @@ 'use strict';
         const executorConfig = config.executor;
+        const defaultPlugin = config.defaultPlugin;
 
@@ -27,3 +28,2 @@ if (!executorConfig || !(Array.isArray(executorConfig)) || executorConfig.length === 0) {
         super();
-        this.defaultExecutor = config.defaultPlugin;
 
@@ -45,5 +45,8 @@ executorConfig.forEach((plugin) => {
 
-            // Set the default executor
-            if (!this.defaultExecutor && !config.defaultPlugin) {
+            if (defaultPlugin && defaultPlugin === plugin.name) {
+                // set the default plugin to the desired one
                 this.defaultExecutor = executorPlugin;
+            } else if (!defaultPlugin && !this.defaultExecutor) {
+                // When given no default plugin, use the first module we instantiated
+                this.defaultExecutor = executorPlugin;
             }
@@ -50,0 +53,0 @@

package.json

 {
   "name": "screwdriver-executor-router",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "A generic plugin for routing builds to specified executors",
@@ -5,0 +5,0 @@ "main": "index.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

7705

increased by3.04%

Lines of code

65

increased by4.84%

No dependency changes