Merge pull request #1430 from webmaster128/allow-commonjs-es6

fix: Don't imply --wrap es6 from --es6 in pbjs
protobufjs · Jun 26, 2020 · b6abd93 · b6abd93
2 parents cd4aeda + 6698d59
commit b6abd93
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/cli/pbjs.js b/cli/pbjs.js
@@ -194,9 +194,8 @@ exports.main = function main(args, callback) {
         return resolved;
     };
 
-    // Use es6 syntax if not explicitly specified on the command line and the es6 wrapper is used
-    if (argv.wrap === "es6" || argv.es6) {
-        argv.wrap = "es6";
+    // `--wrap es6` implies `--es6` but not the other way around. You can still use e.g. `--es6 --wrap commonjs`
+    if (argv.wrap === "es6") {
         argv.es6 = true;
     }