-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Upgrade wp-prettier to v3.0.3 (final) #54775
Conversation
Size Change: 0 B Total Size: 1.62 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍 🚀
@@ -6,3 +6,9 @@ f63053cace3c02e284f00918e1854284c85b9132 | |||
|
|||
# Prettier upgrade to 2.6.2. | |||
33d84b036592a5bf31af05b7710f3b2b14163dc4 | |||
|
|||
# Prettier upgrade to 2.8.5. | |||
c56e8a1910ed74f405b74bbb12fe81dea974e5c3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding it retroactively 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's actually the only way to do it, because we need to have the commit sha, which isn't available until we land it.
Yes, that could be caused by the bad peer dependency. Instead of I pushed a commit where the peer deps are updated to v3. |
I did some local testing of the #54749 issue, and although I can't be 100% sure until the new package versions are actually published on NPM and can be installed from NPM, I'm fairly confident that fixing the peer dependencies is all we need to resolve the issue. |
29e5416
to
8d81651
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for the follow-up.
Fixes #54749.
Finalizes the Prettier upgrade to 3.0.3. After a week without discovering any new bugs, I released final 3.0.3 version of
wp-prettier
(previous releases were betas), and updated Gutenberg'spackage.json
files to use the final version.I also added the reformatting commits for 3.0.3 (#54539) and 2.8.5 (#49258) to the
.git-blame-ignore-revs
file.