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

chore: bump prettier and eslint-plugin-prettier #20181

Merged
merged 3 commits into from Jul 12, 2023

Conversation

mrazauskas
Copy link
Contributor

@mrazauskas mrazauskas commented Jul 12, 2023

Closes #20082
Closes #20179

prettier and eslint-plugin-prettier should be updated together. Since this is major version of prettier, there are few changes in formatting.

@mrazauskas mrazauskas requested a review from a team as a code owner July 12, 2023 05:07
@mrazauskas mrazauskas requested review from Jolg42 and aqrln and removed request for a team July 12, 2023 05:07
@socket-security
Copy link

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: prettier@2.8.8

@codspeed-hq
Copy link

codspeed-hq bot commented Jul 12, 2023

CodSpeed Performance Report

Merging #20181 will degrade performances by 45.68%

Comparing mrazauskas:chore-bump-prettier (3be7ac0) with main (b0ad481)

Summary

❌ 1 regressions
✅ 3 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark main mrazauskas:chore-bump-prettier Change
typescript compilation ~50 Models 87.5 ms 161.1 ms -45.68%

Copy link
Member

@Jolg42 Jolg42 left a comment

Choose a reason for hiding this comment

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

Thanks @mrazauskas for the PR 💚

@Jolg42 Jolg42 added this to the 5.1.0 milestone Jul 12, 2023
@Jolg42 Jolg42 merged commit d9eb856 into prisma:main Jul 12, 2023
57 of 60 checks passed
@mrazauskas mrazauskas deleted the chore-bump-prettier branch July 12, 2023 08:37
@mrazauskas
Copy link
Contributor Author

mrazauskas commented Jul 12, 2023

@Jolg42 Sorry, I forgot one detail. Current, if Prettier v3 is installed, Jest fails to update inline snapshots. There are two solutions:

  • downgrade Prettier to v2
  • add prettierPath: null to Jest config. This will make sure Jest does not run Prettier, but new inline snapshots should be formatted outside Jest.

What would you prefer?

As you see in CI, all works with existing snaps. The problem accrues if they have to be updated or created.

Reference: jestjs/jest#14305

@Jolg42
Copy link
Member

Jolg42 commented Jul 12, 2023

Ouch 😢

It sounds to me like reverting is easier/best until this is figured out then.

@Jolg42
Copy link
Member

Jolg42 commented Jul 12, 2023

PR created from "Revert" UI button #20182

@mrazauskas
Copy link
Contributor Author

It sounds to me like reverting is easier/best until this is figured out then.

Agreed. I will come back after the issue is fixed on Jest side.

@Jolg42
Copy link
Member

Jolg42 commented Jul 12, 2023

Thank you!

@Jolg42
Copy link
Member

Jolg42 commented Jul 27, 2023

Now Jest will error in next version
jestjs/jest#14367

Tracking issue to monitor
jestjs/jest#14305

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