-
-
Notifications
You must be signed in to change notification settings - Fork 237
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
Import footgun: Default export versus named export (No output when used via API) #399
Comments
You've raised an excellent point! |
Might be something for |
Hey! This is now fixed in v9.0.0. |
I got bit by accidentally importing as
import { concurrently } from 'concurrently';
instead ofimport concurrently from 'concurrently';
. The default export applies all the default controllers and logger, whereas the named export does not (the default export uses the "unconfigured" named export to do it's thing).It might be worth adding a note to the README about this or possibly renaming the named export to make it harder to fall into this trap.
The symptoms are that no program output appears, even though it does when you use the concurrently CLI- this is of course because there is no Logger or LogOutput/LogError controllers added in the configuration.
I ended up having to dig into the source code to figure this out.
At the very least, this issue should provide some help for anyone else who hits this footgun.
The text was updated successfully, but these errors were encountered: