-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
BaklavaJS 1.5.2 Build Error #57
Comments
Yup, forgot to add |
It looks like you also forgot |
They shouldn't be needed as they will be included in the webpack-compiled build artifact. Also I can't reproduce this problem in my test projects. Can you post the error log, please? |
If I set |
Ok yeah this happens when you use the option plugin (and probably also the renderer plugin) in TypeScript projects that don't use the For now, as a workaround, just add |
Got it. I'm putting this here because I think it's related to the Webpack build changes. Before 1.5.x, I could do |
True. This happened because I put all the events stuff in a separate package. But as you already wrote, this should change with the build process update. Thanks for mentioning it, I will try to make it more ergonomic when I rewrite the build stuff. |
The build system has completely changed in V2, so these issues shouldn't exist there anymore |
I just updated BaklavaJS to the latest version and now nothing builds. I think something is wrong with the build/dependency configuration, especially with regards to
@baklavajs/events
. A reproducer that shows the same error has been attached. repro.zipThis is my package.json.
repro.zip
Error Message:
The text was updated successfully, but these errors were encountered: