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

fix: ensure isStaticBlock method is available #14259

Closed
wants to merge 1 commit into from

Conversation

imyelo
Copy link

@imyelo imyelo commented Feb 10, 2022

Q                       A
Fixed Issues? Fixes #14230
Patch: Bug Fix? Yes
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

related to #14230 and #14231

@babel-bot
Copy link
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/51222/

@@ -53,7 +53,7 @@ export default declare(({ types: t, template, assertVersion }) => {
}
}
for (const path of body) {
if (!path.isStaticBlock()) continue;
if (!path.isStaticBlock?.()) continue;
Copy link
Member

Choose a reason for hiding this comment

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

This plugin requires Babel 7.12+, and in those versions it's always available.

Copy link
Author

@imyelo imyelo Feb 10, 2022

Choose a reason for hiding this comment

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

This plugin requires Babel 7.12+, and in those versions it's always available.

came across a use case that still triggers exceptions. this repo reproduces the issue:
https://github.com/imyelo/performance-compare/tree/c73d9f8b26540848f29df67b77a0f64311c820f2

Copy link
Author

Choose a reason for hiding this comment

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

emm..
not reproduced in gitpod, seems to be a problem with the local environment itself.

I'm closing this PR, sorry for the intrusion.

@nicolo-ribaudo nicolo-ribaudo added i: regression PR: Bug Fix 🐛 A type of pull request used for our changelog categories labels Feb 10, 2022
@imyelo imyelo closed this Feb 10, 2022
@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label May 13, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
i: regression outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Bug Fix 🐛 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: TypeError: _core.types.isStaticBlock is not a function
3 participants