-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Sync packages for WP 6.9 beta 3 #10453
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
|
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| * @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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 can be done in a follow-up PR for Gutenberg, along with the change mentioned below.
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we use wp_remove_surrounding_empty_script_tags here? cc. @westonruter
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.
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.