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

Replace warning and invariant with tiny versions #634

Merged
merged 2 commits into from
Oct 30, 2018
Merged

Replace warning and invariant with tiny versions #634

merged 2 commits into from
Oct 30, 2018

Conversation

TrySound
Copy link
Contributor

@TrySound TrySound commented Oct 29, 2018

warning and invariant packages are jealously maintained as mirrors
of fbjs versions. Guys do not accept optimizations like esm.

But happily there are alternatives. They are even more lightweight and
save for us 2kB of parsed (minified) code.

@TrySound
Copy link
Contributor Author

/cc @mjackson

@mjackson
Copy link
Member

The build is broken. I'm trying to fix it now.

@TrySound
Copy link
Contributor Author

Wait. I found one problem with them. Names are not provided and they are appear in bundle as function index() {}. Fixing.

@TrySound
Copy link
Contributor Author

`warning` and `invariant` packages are jealously maintained as mirrors
of fbjs versions. Guys do not accept optimizations like esm.

But happily there are alternatives. They are even more lightweight and
save as 2kB of parsed (minified) code.
@TrySound
Copy link
Contributor Author

Done. Updated to latest versions.

@mjackson mjackson merged commit 499bef5 into remix-run:master Oct 30, 2018
@mjackson
Copy link
Member

Thanks, @TrySound. This is good stuff. Makes me wanna do the same thing in all my repos!

@TrySound TrySound deleted the dev-expressions branch October 30, 2018 15:40
@lock lock bot locked as resolved and limited conversation to collaborators Dec 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants