feat(bin): two bins (shell, non-shell) #104
Conversation
Codecov Report
@@ Coverage Diff @@
## next #104 +/- ##
===================================
Coverage 100% 100%
===================================
Files 3 3
Lines 45 42 -3
===================================
- Hits 45 42 -3
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! Thank you! Could we add docs in the README? We could really use an improvement the I think.
I've moved this to go to the |
Thanks. I'll get to work on the documentation over the week-end. |
Revert the default bin (cross-env) to its v3 behavior (not using the shell option). Add a new (cross-env-shell) which uses the shell option. BREAKING CHANGE: Scripts using quotes or escape sequences will see a difference in behavior. Switching to the second bin should resolve any issue. Closes kentcdodds#99.
Documentation updated. What do you think? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one little typo, then we're good. Feel free to merge once you've fixed the typo 👍
README.md
Outdated
second one uses the `shell` option from Node's `spawn`. | ||
|
||
The main use case for `cross-env-shell` is when your need an environment | ||
vartiable to be set across an entire inline shell script, rather than just one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vartiable
should be:
variable
} | ||
} | ||
``` | ||
|
||
The rule of thumb is: if you want to pass to `cross-env` a command that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great to add this. Thanks 👍
I fixed the typo. I'll release a beta for the |
Revert the default bin (cross-env) to its v3 behavior (not using the shell option). Add a new (cross-env-shell) which uses the shell option. BREAKING CHANGE: Scripts using quotes or escape sequences will see a difference in behavior. Switching to the second bin should resolve any issue. Closes #99.
Revert the default bin (cross-env) to its v3 behavior (not using the shell option). Add a new
(cross-env-shell) which uses the shell option.
BREAKING CHANGE: Scripts using quotes or escape sequences will see a difference in behavior.
Switching to the second bin should resolve any issue.
Closes #99.
See prior discussion in #99 et and #102.