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(core): prettier 3 shouldn't cause errors due to esm + compile cache #19042

Merged

Conversation

AgentEnder
Copy link
Member

Current Behavior

Folks receive a weird invalid host options received error when trying to run prettier v3 or other ESM packages during generators or format

Expected Behavior

These errors don't happen.

Related Issue(s)

Fixes #18974
Fixes #18721

@AgentEnder AgentEnder requested a review from a team as a code owner September 6, 2023 21:36
@vercel
Copy link

vercel bot commented Sep 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Sep 6, 2023 9:37pm

@AgentEnder AgentEnder force-pushed the fix/prettier3-invalid-host-options branch from 4c28925 to 2786a8f Compare September 6, 2023 21:36
@pongells
Copy link
Contributor

I am on 16.8.1 using Yarn2 with PNP, fix does not seem to work, I still have to add DISABLE_V8_COMPILE_CACHE=1

@AgentEnder
Copy link
Member Author

Yeah, there was a follow up PR that tidies up a bit of this thats not been released just yet.

@sauldeleon
Copy link

Can confirm that applying this fix solves the issue for me. Thanks!

@github-actions
Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants