Skip to content

Update babel and webpack#2005

Closed
mohd-akram wants to merge 1 commit intohandlebars-lang:masterfrom
mohd-akram:update-babel-webpack
Closed

Update babel and webpack#2005
mohd-akram wants to merge 1 commit intohandlebars-lang:masterfrom
mohd-akram:update-babel-webpack

Conversation

@mohd-akram
Copy link
Copy Markdown
Contributor

@mohd-akram mohd-akram commented Sep 19, 2023

Remove wildcard and require tests since the library has a default export and that's what should be used for browser code.

This is a prerequisite to #2004 and other potential enhancements.

@mohd-akram mohd-akram marked this pull request as draft September 19, 2023 21:35
@mohd-akram
Copy link
Copy Markdown
Contributor Author

Node.js 12, 14 and 16 should be dropped from the CI for pushes and pull requests against master since they're all EOL. They can be kept for pushes and pull requests against the 4.x branch only.

@mohd-akram mohd-akram marked this pull request as ready for review September 19, 2023 21:54
@mohd-akram mohd-akram force-pushed the update-babel-webpack branch 2 times, most recently from 5dbbce0 to 5fcac11 Compare September 20, 2023 12:29
@jaylinski
Copy link
Copy Markdown
Member

The plan is to use https://github.com/developit/microbundle and get rid of grunt, babel and webpack.

If you wanna give it a try, please go ahead, otherwise please close it and wait for me to make the change.

@mohd-akram
Copy link
Copy Markdown
Contributor Author

mohd-akram commented Sep 20, 2023

Is it possible to merge this in the meantime until a switch to microbundle is made, since I already have it working with minimal changes and this is a blocker to other improvements. Also, looking at microbundle, it does not seem to be very regularly updated, and I'm not sure how flexible it is. If, for example, we want a bundle with some polyfills (I have a version of this PR for 4.x with all that, but it hit a snag with AMD support). I don't love grunt or babel, and would rip them out too, but that's a more complex endeavor at this point.

EDIT: Apparently it depends on Babel and rollup itself, and quite a few other dependencies.

Remove wildcard and require tests since the library has a default export and
that's what should be used for browser code.
@jaylinski
Copy link
Copy Markdown
Member

We'll add modern tooling soon. Closing this, since it is almost 3 years old and outdated.

@jaylinski jaylinski closed this Mar 1, 2026
@mohd-akram mohd-akram deleted the update-babel-webpack branch March 26, 2026 07:38
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.

2 participants