Skip to content

switch to babel transform runtime Fixes #152 #161

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

Merged
merged 1 commit into from
Sep 4, 2017

Conversation

graingert
Copy link

No description provided.

@graingert graingert force-pushed the babel-transform-runtime branch from add33f5 to acd9f9d Compare September 4, 2017 11:16
@graingert
Copy link
Author

3.5.0 broke this package in IE because Object.assign isn't variable. You need to enable transform-runtime to automatically shim the feature.

@graingert graingert force-pushed the babel-transform-runtime branch from acd9f9d to 90c9f07 Compare September 4, 2017 11:18
@graingert graingert changed the title switch to babel transform runtime switch to babel transform runtime Fixes #152 Sep 4, 2017
@graingert
Copy link
Author

Fixes #152

@graingert graingert closed this Sep 4, 2017
@graingert graingert reopened this Sep 4, 2017
@graingert
Copy link
Author

@deini ^

@deini
Copy link
Collaborator

deini commented Sep 4, 2017

Looks good, can you remove the /dist changes please? We will push them when we cut a new release.

This allows the use of Object.assign
@graingert graingert force-pushed the babel-transform-runtime branch from 90c9f07 to 7fbdc10 Compare September 4, 2017 20:13
@graingert
Copy link
Author

@deini rebased and fixed. Looking forward to getting the dist files git-ignored in v4

@deini
Copy link
Collaborator

deini commented Sep 4, 2017

Me too @graingert , thanks for updating this.

@deini deini merged commit 8464d92 into angular-redux:master Sep 4, 2017
@graingert graingert deleted the babel-transform-runtime branch September 4, 2017 21:46
@graingert
Copy link
Author

@deini any chance of a release?

@deini
Copy link
Collaborator

deini commented Sep 4, 2017

@graingert I don't have npm access yet. I'll bug @AntJanus tomorrow :)

@graingert
Copy link
Author

or @wbuchwalter

@AntJanus
Copy link
Collaborator

AntJanus commented Sep 5, 2017

@deini I'm about to do a release. Thanks! :)

@graingert
Copy link
Author

@AntJanus @deini it looks like transform-runtime didn't run when transpiling

@graingert
Copy link
Author

@AntJanus @deini ok I've got a fix here: #162

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