-
Notifications
You must be signed in to change notification settings - Fork 44
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
Bumping tested up to version. #43
Conversation
@helen we'll need to update this to merge into |
I went ahead and pushed the same change to |
@jeffpaul Should be okay to update the workflow file in master and it'll run when merging that :) |
@jeffpaul .org only requires that we bump majors :) |
@helen I'm aware of that, my head-scratcher was that our readme.txt states 5.4 while the plugin page on dotorg shows 5.4.1. I'm not certain how dotorg inferred 5.4.1 from our stated 5.4? |
@jeffpaul Right, we only bump majors, and .org assumes/displays the latest minor in that major branch |
Huh, ok. That's a bit odd and potentially concerning as theoretically there could be an conflict between a plugin and a WP minor, especially now that minor releases can include enhancements, that would only be caught in specific testing against a minor versus against a major release. |
Bumping WP tested up to version to 5.4. I followed the same verification process as here.
For #42