-
Notifications
You must be signed in to change notification settings - Fork 65
Bump WordPress tested up to 6.9 and minimum to 6.8 #1024
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
Conversation
|
|
||
| * PHP 7.4+ | ||
| * [WordPress](http://wordpress.org) 6.7+ | ||
| * [WordPress](http://wordpress.org) 6.8+ |
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.
@peterwilsoncc an example of an odd place for a WP version to consider when approaching scripted handling of these automated updates
| }, | ||
| "audit": { | ||
| "ignore": { | ||
| "PKSA-dxyf-6n16-t87m": "Latest version of aws/aws-sdk-php that we can use as newer versions drop support for PHP 7.4" |
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.
@dkotter could be worth considering bumping the PHP min this year
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.
I agree. Probably fairly easy case to make on bumping that to something like 8.3 or 8.2
| <exclude-pattern>*/node_modules/*</exclude-pattern> | ||
| <exclude-pattern>*/vendor/*</exclude-pattern> | ||
|
|
||
| <config name="minimum_supported_wp_version" value="6.1"/> |
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.
🤦🏼 @peterwilsoncc another example of an odd place for a WP min bump to automate
peterwilsoncc
left a comment
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.
Description of the Change
Note
The Plugin Check workflow is failing but that's not due to changes here. I've opened #1023 to fix those.
How to test the Change
Verify all checks pass on this PR
Changelog Entry
Credits
Props @dkotter
Checklist: