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

Temporarily pin Node 20 to 20.5 to avoid Babel error #2373

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Sep 6, 2023

It looks like CI is failing due to an issue with Babel.js conflicting with Node 20.6:

babel/babel#15927
vitejs/vite#14299

For now, let's try pinning Node 20.x to 20.5 until the issue gets fixed in Babel.

Comparable typespec-azure repo PR: https://github.com/Azure/typespec-azure/pull/3502

Issue https://github.com/Azure/typespec-azure/issues/3504 tracks the work to un-pin the Node version.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2023

Changes in this PR will be published to the following url to try(check status of TypeSpec Pull Request Try It pipeline for publish status):
Playground: https://cadlplayground.z22.web.core.windows.net/prs/2373/

Website: https://cadlwebsite.z1.web.core.windows.net/prs/2373/

@daviwil
Copy link
Contributor Author

daviwil commented Sep 6, 2023

@mikeharder Can you take a look to see whether I'm pinning the Node version correctly in the 20.x jobs? We've hit an issue with a dependency conflicting with Node 20.6 so I'm trying to pin it back to 20.5.

Nevermind, figured it out :)

@daviwil daviwil merged commit e561be5 into microsoft:main Sep 6, 2023
@daviwil daviwil deleted the fix-babel-error branch September 6, 2023 13:43
@mikeharder
Copy link
Contributor

This appears to be a regression in Node 20.6.0: nodejs/node#49497. I agree the best workaround is pinning to 20.5. Will likely be fixed in 20.6.1.

@mohanvaidya
Copy link

I have node v 20.5.1 installed on my machine still fails to run...! why..?
image

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.

4 participants