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: babel-code-frame to version 7 #10808
Conversation
Hi, thanks for the PR. Do you mind changing the commit title to start with |
My bad. Fixed! Looks like GitHub has issues updating PRs right now, might be a bit until this shows up correctly. |
This LGTM. Thanks for contributing! |
Ah, actually, just had a thought. I wonder if this will interfere with the release process at all since this version won't match the Babel version we use for our browser bundle. Can you confirm this works when running the bundling step? |
Running Any idea why the |
|
🤦♂️ Fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from the commit message typo this LGTM. Thanks for contributing!
@realityking It doesn't look like the change made it to GitHub |
I've pushed a commit with the correct message but GitHub has some trouble updating PRs. Might be a bit until it shows up. |
Now GitHub caught up :) |
Could someone restart the Travis build? Something timed out on the Node,js 6 build. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[X] Other, please explain:
Bumping babel-code-frame to version 7. Note I didn't touch the babel version used in the build process, only code-frame.
What changes did you make? (Give an overview)
Is there anything you'd like reviewers to focus on?