Skip to content

Switch bundler from ESBuild to Webpack #1911

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

Closed
wants to merge 3 commits into from

Conversation

koesie10
Copy link
Member

In terms of keeping down the number of dependencies and different tools we are using, this switches the bundler for the main extension module from ESBuild to Webpack to keep it in line with the bundler used for the webview.

⚠️ This is based on #1799 and is just for testing whether this also works (for now).

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@koesie10 koesie10 force-pushed the koesie10/bundle-extension branch 2 times, most recently from 08549c7 to 6bf19eb Compare January 11, 2023 09:30
In terms of keeping down the number of dependencies and different tools
we are using, this switches the bundler for the main extension module
from ESBuild to Webpack to keep it in line with the bundler used for the
webview.
This reduces the output size by taking into account whether a release
build is being created.
@koesie10 koesie10 force-pushed the koesie10/bundle-extension-webpack branch from 085c112 to bf86aa9 Compare January 11, 2023 10:04
Base automatically changed from koesie10/bundle-extension to main January 13, 2023 12:13
@koesie10
Copy link
Member Author

For now, let's see how well ESBuild works. If we run into any problems or want to unify the two builds, we can think about switching this to Webpack.

@koesie10 koesie10 closed this Jan 13, 2023
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