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: Bundle errors #72

Merged
merged 2 commits into from
Oct 17, 2019
Merged

fix: Bundle errors #72

merged 2 commits into from
Oct 17, 2019

Conversation

tassoevan
Copy link
Collaborator

  1. React was declared as an external AMD dependency instead of react;
  2. Some build systems prefer .mjs over .js without proper parsing;
  3. MiniCssExtractPlugin.loader cannot be used with 'style-loader' in the same chain (see https://github.com/webpack-contrib/mini-css-extract-plugin#advanced-configuration-example)

@tassoevan tassoevan added the 🐛 Bug Something isn't working label Oct 17, 2019
@tassoevan tassoevan requested a review from ggazzo October 17, 2019 17:49
@ggazzo ggazzo merged commit 77f5c12 into develop Oct 17, 2019
@ggazzo ggazzo deleted the fix/webpack branch October 17, 2019 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants