[PATCH] Be robust when "named argument" doesn't exist - Githubissues

MarkForged / nps

NPM Package Scripts -- All the benefits of npm scripts without the cost of a bloated package.json and limits of json

MIT License

6 stars 0 forks source link

[PATCH] Be robust when "named argument" doesn't exist #3

Closed mf-bruce closed 2 years ago

mf-bruce commented 2 years ago

Summary The issue with the way positional args was defined is that the format conflicts with environment variables and nps was throwing an error when an environment variable was used, eg. consider a script that uses $FOO. When a token with the $ prefix is not found in the args, just ignore it and continue.

Test Plan

Updated tests

What:

Why:

How:

Checklist:

[ ] Documentation
[ ] Tests
[ ] Ready to be merged
[ ] Added myself to contributors table