-
Notifications
You must be signed in to change notification settings - Fork 1
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
Bundler generalization #43
Comments
Oh yes, when The way I see it the bundler functionality can be abstracted, and when any bundler specifics are fed as scripts that are just called to be executed by the core plugin, then any bundler can be plugged in without overcomplicating the core plugin implementation. |
I guess the question then is whether it would make more sense to just work from |
Once we refine the full scope of work from the issues you created it will be easier to determine how to proceed. But |
Ok looking at your responses to prior #39 and #41 I think it looks like what we have in mind is different enough that I'm going to keep working on |
Closing this because the suggestion will be addressed in a separate project. |
When I saw that you had started working on a
scalajs-esbuild
plugin, it occurred to me that there is a lot of functionality here that is independent of the particular bundler used. It might be worth abstracting that functionality into a base plugin that is extensible in the manner of, say,jsEnv
. There are at least two compelling reasons to do so:esbuild
for tests and dev servers, where I don't really care so much how the code is bundled as that it does so quickly, but usewebpack
for production bundles because there are some very specific configurations I need to this optimally.For reference I already have POC for this idea here, much of which is based on
scalajs-vite
. It works fairly well so far (I'm using it in my project), but I'm not married to it andscalajs-vite
is both cleaner and reveals a deeper understanding ofsbt
than I have. Any interest in taking an approach like that?The text was updated successfully, but these errors were encountered: