-
Notifications
You must be signed in to change notification settings - Fork 135
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
[Question] beforeEmit not default as true? #97
Comments
I agree it would be better to have it as default, though changing it in its current state could break people depending on it not being default. I will definitely change this for future versions though. |
This has been changed in |
Sweet, thanks for the update. Keep up the good work 😄 |
Awesome that this default got changed! However, it wasn't quite clear for me that this was the case in the 2.0 release notes - perhaps stick it in there, if possible? Thank you @chrisblossom and @johnagan. The defaults should work well for most use cases now - including mine 🎉 |
@oles I've added Additional v2 Information to further clarify the changes in |
Hi Dev,
Just a question as to why the beforeEmit option doesn't default to true.
Seems like that would be the logical thing to do?
Great Plugin BTW
AAllport
The text was updated successfully, but these errors were encountered: