Using switch-case instead of ifs

Fixes #5920
elastic · Jan 16, 2016 · f3df973 · f3df973
1 parent ef68493
commit f3df973
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/src/cli/plugin/plugin.js b/src/cli/plugin/plugin.js
@@ -19,16 +19,17 @@ export default function pluginCli(program) {
 
     const logger = pluginLogger(settings);
 
-    if (settings.action === 'install') {
-      installer.install(settings, logger);
+    switch (settings.action) {
+      case 'install':
+        installer.install(settings, logger);
+        break;
+      case 'remove':
+        remover.remove(settings, logger);
+        break;
+      case 'list':
+        lister.list(settings, logger);
+        break;
     }
-    if (settings.action === 'remove') {
-      remover.remove(settings, logger);
-    }
-    if (settings.action === 'list') {
-      lister.list(settings, logger);
-    }
-
   }
 
   program