Skip to content

Transpile coffeescript written in ES6 #35

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 5 commits into from
Dec 13, 2017

Conversation

wataruoguchi
Copy link
Contributor

coffeescript can adopt babel as a transpiler. transpile is the way how they set transpiler.

Please see http://coffeescript.org/#transpilation

This allows you to have vue files which adopt coffee/coffeescript (ES6) as lang.
I added tests and vue files.

sourceMap: true
sourceMap: true,
transpile: {
presets: ['env']
Copy link
Member

Choose a reason for hiding this comment

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

Would we want to use presets that the user has in their babelrc?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@eddyerburgh Yes, actually that should be better

@wataruoguchi
Copy link
Contributor Author

@eddyerburgh Thanks for reviewing the previous commit. These changes should allow users to use their .babelrc

Copy link
Member

@eddyerburgh eddyerburgh left a comment

Choose a reason for hiding this comment

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

Looks great, could you add some tests to the test/coffee.spec.js that checks it uses the babelrc. You can see similar tests in Babel.spec.js

@wataruoguchi
Copy link
Contributor Author

@eddyerburgh Here ^ some tests added. Please review it. Pleasure to contribute to this project :)

@eddyerburgh eddyerburgh merged commit c5448c7 into vuejs:master Dec 13, 2017
@eddyerburgh
Copy link
Member

Thanks @wataruoguchi 🙂

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