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 to babel 7 #2

Merged
merged 1 commit into from
Sep 9, 2018
Merged

Update to babel 7 #2

merged 1 commit into from
Sep 9, 2018

Conversation

DeMoorJasper
Copy link
Contributor

There has been some changes in the typings package. So I updated that package to v7.
While I was at it, I also added an engines field in package.json and updated the internally used babel to 7 so builds go slightly faster and benefit from all new bugfixes.

This is a breaking change, as support for Babel 6 will be dropped.

@TimothyGu TimothyGu merged commit 6ff8db7 into pugjs:master Sep 9, 2018
@DeMoorJasper
Copy link
Contributor Author

DeMoorJasper commented Sep 18, 2018

@TimothyGu When will this be pushed to npm?

@joaovieira
Copy link

joaovieira commented Oct 8, 2018

Yes, please. Can we publish it? We need this for JSX's new <> fragment syntax.

@joaovieira
Copy link

ping @TimothyGu ☝️

@joaovieira
Copy link

@TimothyGu, again, can we publish this? It's a dependency of https://github.com/fraserxu/babel-jsxgettext and without this PR we can't parse JSX trees that include the new Fragment syntax <>. It's causing many headaches that could be avoided if we just published this.

@AxelDelmas
Copy link

Same here. Can you publish it on npm, please?

@AxelDelmas
Copy link

I made the builds available on github, if you're blocked you can change your package.json to install babylon-walk from this repo: https://github.com/streamroot/babylon-walk (this fork won't be maintained at all though, it's just a temporary solution before the package is updated on npm)

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.

4 participants