-
Notifications
You must be signed in to change notification settings - Fork 12
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 ember-cli-babel
to version 7
#14
base: master
Are you sure you want to change the base?
Conversation
Honestly I haven't used this plugin in many years. If you are using it and it still works for you with this change I'm happy to merge and release... |
Although it looks like Travis is unhappy because:
|
I was just trying to figure that out. Any ideas what that is and why it fails? |
Actually - I think the real error is:
I set my node and yarn versions to the same as TravisCI via volta ( Maybe you can update |
Done. But this means a new major version. And while we're at it, we could do other version updates I guess? |
0381c4d
to
9046f48
Compare
9046f48
to
124bb20
Compare
@vitch - this is ready for merging. I moved a few things around and updated the used I tried out the branch, it seems to work great. So a new (major) release will be appreciated! :) Thanks! |
This is the only plugin that I use that still uses
ember-cli-babel
6 so I decided to update it. Not sure if this is enough or more changes are needed to the plugin?cc @vitch