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 Babel parser to 7.9.4 #118

Merged
merged 3 commits into from Jun 30, 2020

Conversation

OliverJAsh
Copy link
Contributor

This seemed to fix the bug I mentioned over Twitter DMs, whereby loc was null for the call expression in export default a(1);.

I chose 7.9.4 because:

  • that's the version used on AST explorer and I wasn't able to reproduce the bug there
  • I had type errors when I tried to upgrade to the latest versions (7.10.x)

@@ -227,6 +227,11 @@
dependencies:
"@babel/types" "^7.8.3"

"@babel/helper-validator-identifier@^7.10.4", "@babel/helper-validator-identifier@^7.9.0":
version "7.10.4"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ahh I have no idea why we're getting 7.10 here. Yarn deduping is the worst.

Copy link
Owner

Choose a reason for hiding this comment

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

No idea! As long as tests are green and bug is fixed, that's legit to me.

@nicoespeon
Copy link
Owner

I hoped that was a bug with the parser and that an upgrade would fix it. Thanks for doing that 👍

I can merge confidently since CI is all green 👌

@nicoespeon
Copy link
Owner

@all-contributors add OliverJAsh for ideas,bug,code,doc

(I'm sure it won't work, I'll fix this so it works again)

@allcontributors
Copy link
Contributor

@nicoespeon

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

@nicoespeon nicoespeon merged commit 82df32f into nicoespeon:master Jun 30, 2020
@nicoespeon
Copy link
Owner

@all-contributors add OliverJAsh for ideas,bug,code,doc

@allcontributors
Copy link
Contributor

@nicoespeon

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

@nicoespeon
Copy link
Owner

@all-contributors please add OliverJAsh for ideas, bug, code and doc

@allcontributors
Copy link
Contributor

@nicoespeon

I've put up a pull request to add @OliverJAsh! 🎉

@OliverJAsh OliverJAsh deleted the maintenance/upgrade-babel branch July 1, 2020 09:32
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.

None yet

2 participants