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

update dependencies #94

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

romainmenke
Copy link
Member

@romainmenke romainmenke commented Dec 14, 2024

With the current setup using npm workspaces for the polyfills I didn't take build tools for polyfills into account as we didn't have any. And polyfills tend to be self contained and have no dependencies.

However URL and IteratorHelpers now have a build tool to prepare the polyfills for older browsers.

Build tools always have incredibly deep and complex dependency graphs.

A side effect of introducing build tools for polyfills is that running npm update now has unwanted side effects as it will update those deeply nested dependencies even when direct dependency (e.g. browserify, babelify, ...) has a pinned version.

I expect this PR to change, even when it contains a few minor changes to transitive dependencies. This shows the brittleness of the current approach.


I am also a bit concerned that we are giving up a lot of control over the final result and the quality thereof.

These tools are known for adding a bunch of bloat which often isn't needed.

Manually written code for older browsers, all in a single file, will always be better and smaller. Ideally we don't have any build tools for polyfills.

I know that this is more work and that we then lose the connection to upstream maintainers. But I do think this is best for end users.

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