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

Bundle UMD with rollup #449

Merged
merged 5 commits into from
Feb 13, 2019
Merged

Bundle UMD with rollup #449

merged 5 commits into from
Feb 13, 2019

Conversation

TrySound
Copy link
Contributor

@TrySound TrySound commented Jan 14, 2019

-Webpack production bundle is 18407b (no gzip)
+Rollup production bundle is 14975b (no gzip)

```diff
-Webpack production bundle is 18407b (no gzip)
+Rollup production bundle is 14975b (no gzip)
```
@TrySound
Copy link
Contributor Author

Oops, it's 14975b. Didn't expected such behaviour from babel config. I don't like custom presets for this. They need to be learned.

Copy link
Collaborator

@jquense jquense left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm @silvenon can you also take a look

@TrySound
Copy link
Contributor Author

Guys, I already have esm support locally. Will push it after merging this PR.

@TrySound
Copy link
Contributor Author

Hi @silvenon. Could you take a look at this?

@TrySound
Copy link
Contributor Author

@silvenon Friendly ping

@TrySound
Copy link
Contributor Author

TrySound commented Feb 9, 2019

Sorry for noise. Just want to get things done :)
/cc @silvenon

@TrySound
Copy link
Contributor Author

@silvenon Ping :)

@jquense jquense merged commit ac15233 into reactjs:master Feb 13, 2019
@jquense
Copy link
Collaborator

jquense commented Feb 13, 2019

I think we can move this along, happy to any feedback in retro

@TrySound TrySound deleted the rollup branch February 13, 2019 20:47
errendir added a commit to errendir/react-transition-group that referenced this pull request Feb 14, 2019
…d-compatible

* origin/master:
  chore: bundle UMD with rollup (reactjs#449)
  chore(release): 2.5.3 [skip ci]
  fix: strip custom prop-types in production (reactjs#448)
  chore: fix storybook (reactjs#445)
  chore(release): 2.5.2 [skip ci]
  fix: pass appear to CSSTransition callbacks (reactjs#441)
  docs: Fix type declaration comment (reactjs#439)
  chore(release): 2.5.1 [skip ci]
  fix: prevent calling setState in TransitionGroup if it has been unmounted (reactjs#435)
errendir added a commit to errendir/react-transition-group that referenced this pull request Feb 14, 2019
…d-compatible

* origin/master:
  chore: bundle UMD with rollup (reactjs#449)
  chore(release): 2.5.3 [skip ci]
  fix: strip custom prop-types in production (reactjs#448)
  chore: fix storybook (reactjs#445)
  chore(release): 2.5.2 [skip ci]
  fix: pass appear to CSSTransition callbacks (reactjs#441)
  docs: Fix type declaration comment (reactjs#439)
  chore(release): 2.5.1 [skip ci]
  fix: prevent calling setState in TransitionGroup if it has been unmounted (reactjs#435)
@silvenon
Copy link
Collaborator

silvenon commented Feb 14, 2019

So sorry guys. Thanks a lot @TrySound for your work and thanks @jquense for reviewing ❤️

@jquense
Copy link
Collaborator

jquense commented Feb 26, 2019

🎉 This PR is included in version 2.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants