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 astring #35

Closed
wants to merge 1 commit into from
Closed

fix astring #35

wants to merge 1 commit into from

Conversation

kentcdodds
Copy link

@kentcdodds
Copy link
Author

Alternative solution: #35

@kentcdodds kentcdodds mentioned this pull request Apr 8, 2021
@kentcdodds
Copy link
Author

Hmmm... Not sure what's wrong. Based on what's on npm this should work 🤔

@wooorm
Copy link
Owner

wooorm commented Apr 8, 2021

Heya! we detected a broken patch in astring yesterday (davidbonnet/astring#505), it was fixed 19 hours ago: davidbonnet/astring@6d8a122. So, npm update should do the trick?

And: it was export maps and ESM 😅

@wooorm
Copy link
Owner

wooorm commented Apr 8, 2021

You caught me right before heading out. I’ll take my laptop with me, so do let me know if I need to land a hotfix!

@kentcdodds
Copy link
Author

Ah, gotcha. I'll let folks know that npm update should fix it for them. Thanks!

@wooorm
Copy link
Owner

wooorm commented Apr 11, 2021

I think the fix in astring makes this PR superfluous, so I’ll close these two.

For anyone else stumbling on this, please try npm update, and let me know if that doesn’t fix it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants