MetaMask / snaps-cli

A CLI for developing MetaMask plugins.
ISC License
38 stars 26 forks source link

Make applyConfig transform argv instead of builders #84

Closed rekmarks closed 3 years ago

rekmarks commented 3 years ago

Per the yargs docs, yargs.middleware(...) is intended to be used for argv transformations that should be run before a command is applied. applyConfig used to transform the object exported by builders. This is a bad idea. Instead, it now transform argv in the yargs middleware, just like sanitizeInputs was already doing.