Sync packages for WP 6.9 beta 3#10453
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Unlinked AccountsThe following contributors have not linked their GitHub and WordPress.org accounts: @deemkaaa09-lang. Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases. Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
| * @param string $dropdown_id ID of the dropdown field. | ||
| * | ||
| * @return string Returns the dropdown onChange redirection script. |
There was a problem hiding this comment.
| * @param string $dropdown_id ID of the dropdown field. | |
| * | |
| * @return string Returns the dropdown onChange redirection script. | |
| * @param string $dropdown_id ID of the dropdown field. | |
| * @return string Returns the dropdown onChange redirection script. |
There was a problem hiding this comment.
What's the objective reason to remove the blank line? It passes the coding standard checks, and other PHPDocs in the same file have had the same blank line for years
There was a problem hiding this comment.
I concur with @priethor here, no need to remove that line as most (all?) cases here include the line spacing (and at least for me makes it easier to read those docs/comments).
There was a problem hiding this comment.
It's not a blocker but as per WordPress Inline Documentation Standards for PHP, there should not be an extra line between @param and @return tags.
There was a problem hiding this comment.
This can be done in a follow-up PR for Gutenberg, along with the change mentioned below.
There was a problem hiding this comment.
Agreed that any such change should be done in Gutenberg, starting with updating the PHPCS rules to match the coding standards.
| return wp_get_inline_script_tag( | ||
| trim( str_replace( array( '<script>', '</script>' ), '', ob_get_clean() ) ) . | ||
| "\n//# sourceURL=" . rawurlencode( __FUNCTION__ ) | ||
| ); |
There was a problem hiding this comment.
Should we use wp_remove_surrounding_empty_script_tags here? cc. @westonruter
There was a problem hiding this comment.
Yes, although it's not important. It can be done in a follow-up PR to Gutenberg.
|
@desrosj, I see you are debugging the Windows build failures like in this PR. Is it a blocker to merge patches with failing workflows? |
|
@priethor I did create that PR to rule out a persisting cache as the reason for the failure. But I'm not actively investigating, mainly because I don't have a Windows machine. This is similar to the other issue that happened earlier in the release. It's not a firm blocker, but anyone using Windows to contribute will potentially be unable to do so. |
This comment was marked as spam.
This comment was marked as spam.
This issue was recently fixed in the Gutenberg repository: WordPress/gutenberg#72960
|
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
|
I've done one last update before the Beta3 freeze, and it should be ready to go once GH actions pass. |
Changes can be found at https://github.com/WordPress/gutenberg/commits/wp/6.9/. Developed in #10453. See https://make.wordpress.org/core/handbook/about/release-cycle/block-editor-release-process-for-major-releases/#package-updates-and-core-patches. Fixes #64185. Props priethor, mukesh27, jeffpaul, westonruter, desrosj, wildworks. git-svn-id: https://develop.svn.wordpress.org/trunk@61125 602fd350-edb4-49c9-b593-d223f7449a82
Changes can be found at https://github.com/WordPress/gutenberg/commits/wp/6.9/. Developed in WordPress/wordpress-develop#10453. See https://make.wordpress.org/core/handbook/about/release-cycle/block-editor-release-process-for-major-releases/#package-updates-and-core-patches. Fixes #64185. Props priethor, mukesh27, jeffpaul, westonruter, desrosj, wildworks. Built from https://develop.svn.wordpress.org/trunk@61125 git-svn-id: http://core.svn.wordpress.org/trunk@60461 1a063a9b-81f0-0310-95a4-ce76da25c4cd
Changes can be found at https://github.com/WordPress/gutenberg/commits/wp/6.9/. Developed in WordPress/wordpress-develop#10453. See https://make.wordpress.org/core/handbook/about/release-cycle/block-editor-release-process-for-major-releases/#package-updates-and-core-patches. Fixes #64185. Props priethor, mukesh27, jeffpaul, westonruter, desrosj, wildworks. Built from https://develop.svn.wordpress.org/trunk@61125 git-svn-id: https://core.svn.wordpress.org/trunk@60461 1a063a9b-81f0-0310-95a4-ce76da25c4cd
Trac ticket: https://core.trac.wordpress.org/ticket/64185
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.