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

feat: remove virtual module plugin, much refactoring, webpack 5 support #48

Merged
merged 1 commit into from
Feb 2, 2021

Conversation

DylanPiercey
Copy link
Contributor

BREAKING CHANGE: drops support for Marko 4 compilation

Description

This PR replaces #42.
It adds support for webpack 5, while maintaining webpack 4 support.
Marko 4 support has been dropped.

This PR is a fairly large refactor to simplify the logic and also to remove the usage of the virtual modules plugin which prevented us from supporting webpack 5.

Checklist:

  • I have updated/added documentation affected by my changes.
  • I have added tests to cover my changes.

@DylanPiercey DylanPiercey force-pushed the remove-virtual-modules branch 2 times, most recently from c4a3db9 to 4b745eb Compare January 30, 2021 18:31
@DylanPiercey DylanPiercey mentioned this pull request Jan 30, 2021
2 tasks
@DylanPiercey
Copy link
Contributor Author

This PR is not quite merge ready. Although the tests are passing and it is possible everything works, more testing is needed.

If anyone would like to help with that effort I've released a pre-release (@marko/[email protected]).

@DylanPiercey DylanPiercey force-pushed the remove-virtual-modules branch 6 times, most recently from 7e38097 to 4f1d6e3 Compare February 2, 2021 20:59
@DylanPiercey DylanPiercey force-pushed the remove-virtual-modules branch from 4f1d6e3 to fbd12f3 Compare February 2, 2021 21:00
@DylanPiercey DylanPiercey merged commit 7f0c0db into master Feb 2, 2021
@DylanPiercey DylanPiercey deleted the remove-virtual-modules branch February 2, 2021 22:45
DylanPiercey added a commit that referenced this pull request Feb 2, 2021
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.

1 participant