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

Upgrade to the stable babel 7 release #460

Merged
merged 3 commits into from
Aug 31, 2018
Merged

Upgrade to the stable babel 7 release #460

merged 3 commits into from
Aug 31, 2018

Conversation

bellini666
Copy link
Contributor

@bellini666 bellini666 commented Aug 30, 2018

I ran babel-upgrade in the packages to generate this change.

@oliviertassinari
Copy link
Owner

I'm already working on it in #459.

@bellini666
Copy link
Contributor Author

@oliviertassinari oh, didn't know that! Ok, will abandon this.

Do you have an ETA to when babel7 support will be available at npm? I'm trying to upgrade my project here and this dependency is the only one preventing me from doing that

@oliviertassinari
Copy link
Owner

@hackedbellini I don't have any ETA, it could take me 1-3 days, maybe, I don't know.

@bellini666 bellini666 closed this Aug 30, 2018
@bellini666 bellini666 deleted the babel7 branch August 30, 2018 15:07
@bellini666
Copy link
Contributor Author

@oliviertassinari ok, I'll wait until #459 is done! Thanks

@oliviertassinari
Copy link
Owner

@hackedbellini Would you mind reopening the pull-request so I can continue your effort? Giving you some credit? Thanks.

@bellini666 bellini666 restored the babel7 branch August 30, 2018 15:30
@bellini666 bellini666 reopened this Aug 30, 2018
@bellini666
Copy link
Contributor Author

@oliviertassinari hahaha there you go! Thanks :)

@oliviertassinari oliviertassinari self-assigned this Aug 30, 2018
@oliviertassinari oliviertassinari merged commit 28b3e51 into oliviertassinari:master Aug 31, 2018
@bellini666
Copy link
Contributor Author

I saw that you merged this and released a new version! Can you ping me here when it is available on npm? I just checked and it wasn't yet =P

@oliviertassinari
Copy link
Owner

done :)

@bellini666
Copy link
Contributor Author

@oliviertassinari thanks! Just updated my project, everything working great now :)

@oliviertassinari
Copy link
Owner

@hackedbellini I'm happy to hear it! I have been upgrading a lot of things. It could have went badly :)

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