-
Notifications
You must be signed in to change notification settings - Fork 69
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
Pin WordPress to 6.0.3 for pull request checks. #5043
Conversation
Size Change: 0 B Total Size: 999 kB ℹ️ 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.
This seems a sensible temporary fix to me, unblocking us within the current release code freeze.
As you mentioned in the Slack discussion, e2e tests are still passing on WC 6.1, giving us confidence that critical flows are working. p1667353741152329-slack-CGGCLBN58
Ideally, we can update these failing tests to work with WC 6.1 before the 5.0.0 release.
Update: it looks like this is in progress already 🎉 #5049
Closing in favor of #5049, which simply fixed the issue :) |
Fixes #5042
Changes proposed in this Pull Request
This PR pins WordPress to 6.0.3 for the compatibility, code coverage and PHP linting and tests pull request checks.
Testing instructions
develop
.)npm run changelog
to add a changelog file, choosepatch
to leave it empty if the change is not significant. You can add multiple changelog files in one PR by running this command a few times.Post merge