Skip to content
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 --file not being respected #84

Merged
merged 1 commit into from
Oct 16, 2020
Merged

Fix --file not being respected #84

merged 1 commit into from
Oct 16, 2020

Conversation

willsoto
Copy link
Contributor

What's the problem this PR addresses?

--file not being respected

How did you fix it?

Any files that were added via --file were being blown away when Webpack's entrypoints were being added.
In order to resolve this, the entrypoints now get added via Mocha's addFile method.

Closes #20

Any files that were added via --file were being blown away when Webpack's entrypoints were being added.
In order to resolve this, the entrypoints now get added via Mocha's addFile method.

Closes #20

Signed-off-by: Will <will-soto@pluralsight.com>
@larixer
Copy link
Member

larixer commented Oct 16, 2020

@willsoto Nice, many thanks!

@larixer larixer merged commit ad9fc4f into sysgears:master Oct 16, 2020
@larixer
Copy link
Member

larixer commented Oct 16, 2020

Published in mochapack@2.0.4

@willsoto
Copy link
Contributor Author

Thanks for the quick review and release!

@willsoto willsoto deleted the fix/20 branch October 16, 2020 13:51
@willsoto
Copy link
Contributor Author

@larixer It doesn't seem like the change made it into 2.0.4. I just upgraded but the compiled code does not contain the change I made.

Screen Shot 2020-10-16 at 9 58 10 AM

Screen Shot 2020-10-16 at 9 58 15 AM

@larixer
Copy link
Member

larixer commented Oct 16, 2020

@willsoto Could you check if it came through into 2.0.5 ?

Yarn 2 does not use prepublishOnly script, it uses prepack script instead, I have adjusted that

@willsoto
Copy link
Contributor Author

@larixer - works now. Thank you very much!

@larixer
Copy link
Member

larixer commented Oct 16, 2020

@willsoto Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for --file option
2 participants