-
Notifications
You must be signed in to change notification settings - Fork 5
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
Separate into Opal bootstrap and loader #33
Comments
@cj - any opinion on who "owns" the new project (github and NPM) ? |
Also, looks like https://github.com/anthonny/opal-npm-wrapper is out there, although there are some things that would need tweaking with that. |
I might raise an issue and ask how willing they would be to cover the cases the code in the repo does. |
I don't mind :) Interesting. Let me know if you get a hold of him. Should we just fork it for now? |
No urgent need for me to do anything, I'm content to wait and see how it plays out personally. |
I've been wondering if this loader is really doing 2 things: presenting a "bootstrapped" version of Opal in Node and serving as webpack loader. What if this was split into 2 packages
Opal NPM - could also be called Opal bootstrap. This would include the compiler.rb patches, bundlerCheck/Execute, getRubyMetadata, and the getOpal*Filename.js files
Webpack loader - everything else
This would make it easier to implement #7 because a separate plugin could be developed that could modify webpack's "load path" but take advantage of the Opal NPM code above just like the loader does. No urgency here, but any thoughts @cj ?
The text was updated successfully, but these errors were encountered: