Allow users to pass additional arguments to spawned func host start process when running sls offline
Closes #365
How did you implement it:
Created new option spawnargs (shortcut a) that takes a whitespace separated string containing additional arguments for new process.
IMPORTANT - PR is targeting linux & python support branch because of the significant changes to the way we use core tools within the plugin. That branch has a more forward facing approach to generalizing the use of azure-functions-core-tools and made more sense to branch off of it for this issue.
How can we verify it:
Run
sls offline -a "--cors *"
Rather than func host start being spawned, func host start --cors * is spawned.
Todos:
Note: Run npm run test:ci to run all validation checks on proposed changes
[x] Ensure there are no lint errors. Validate via npm run lint Note: Some reported issues can be automatically fixed by running npm run lint:fix
[x] Write tests and confirm existing functionality is not broken. Validate via npm test
[x] Write documentation
[x] Provide verification config / commands / resources
[x] Enable "Allow edits from maintainers" for this PR
[x] Update the messages below
Is this ready for review?: YES
Is it a breaking change?: NO
What did you implement:
Allow users to pass additional arguments to spawned
func host start
process when runningsls offline
Closes #365
How did you implement it:
Created new option
spawnargs
(shortcuta
) that takes a whitespace separated string containing additional arguments for new process.IMPORTANT - PR is targeting linux & python support branch because of the significant changes to the way we use core tools within the plugin. That branch has a more forward facing approach to generalizing the use of
azure-functions-core-tools
and made more sense to branch off of it for this issue.How can we verify it:
Run
Rather than
func host start
being spawned,func host start --cors *
is spawned.Todos:
Note: Run
npm run test:ci
to run all validation checks on proposed changesValidate via
npm run lint
Note: Some reported issues can be automatically fixed by running
npm run lint:fix
Validate via
npm test
Is this ready for review?: YES Is it a breaking change?: NO