-
Notifications
You must be signed in to change notification settings - Fork 80
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
Bump Min WP Version to 6.4 #1287
base: trunk
Are you sure you want to change the base?
Conversation
} | ||
|
||
if ( | ||
! \function_exists( 'register_block_type_from_metadata' ) || |
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.
let's keep that, so that it will still work with classicpress or the classic editor enabled.
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.
The functions exist with the classic editor enabled. Should ClassicPress be a consideration?
* | ||
* @return array The attachment IDs. | ||
*/ | ||
protected function get_classic_editor_image_attachments( $max_images ) { |
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.
let's keep that, for people that still use the classic editor
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.
Ah, this is only about the attachments 🤔
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.
Good to remove?
Let's bump the min version to 6.4. The last three versions is what the Plugin Directory expects plugins to be tested with and it gets us guaranteed
WP_HTML_Tag_Processor
support. 6.4 is also the first version that supports PHP 8.3.Based on https://fedi.wrm.sr/software/wordpress/versions, 126 installs are below 6.4, or around 1.47% of all ActivityPub installs.
Proposed changes:
Other information: