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

Completely move build system from rollup to webpack #290

Merged
merged 8 commits into from
Sep 4, 2021
Merged

Conversation

yhatt
Copy link
Member

@yhatt yhatt commented Sep 4, 2021

webpack + esbuild build setting for web extension have succeeded in #283. Maintaining settings for two build systems is hard so it's time to move from rollup to webpack. Using esbuild means making faster to build, and we will get a better development experience.

rollup could not bundle Marp Core by circular dependency, but webpack can bundle Marp Core correctly. By this change, we can expect that file I/O for loading the extension would reduce and the time for activation would be faster (Resolve #124).

This moving does not mean bringing to leave from rollup even in other Marp projects. rollup is still the best bundler for npm modules like Marp Core and Marp CLI, and we will continue to use rollup as a primary choice to build and bundle.

@yhatt yhatt merged commit a8c44aa into main Sep 4, 2021
@yhatt yhatt deleted the moving-to-webpack branch September 4, 2021 19:03
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.

Extension causes high cpu load
1 participant