-
-
Notifications
You must be signed in to change notification settings - Fork 236
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: bugfix for wildcard naming #212
fix: bugfix for wildcard naming #212
Conversation
Pull Request Test Coverage Report for Build 372
💛 - Coveralls |
@gustavohenke the coveralls issue seems to be a false positive. |
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.
Nice job, thank you!
I'll loosen the Coveralls integration 😃
Before merging, I thought of checking how v3 would behave.
WDYT? |
Any progress on this? |
@gustavohenke : can this be merged, please ? |
@gustavohenke Thanks so much for Looking forward to the behavior illustrated in the docs which will save precious space in the terminal. concurrently "npm:watch-*"
# Equivalent to:
concurrently -n js,css,node "npm run watch-js" "npm run watch-css" "npm run watch-node" |
Seems like the expected behavior. Would love to see this merged, I was wondering why my console output wouldn't match what is described in the docs. And thanks for the package, of course! |
Builds all failed because of Coverage, but will merge anyway since everything else is passing and I've applied the suggested changes above! |
Published as v6.5.1! |
Fixes #211
Fixes #148
See #211 for more info and a repro case.