-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Update @wordpress npm packages. #626
Update @wordpress npm packages. #626
Conversation
@@ -0,0 +1,196 @@ | |||
<?php |
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 move this file to src/wp-includes/block-supports/class-wp-block-supports.php
.
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 actually reverted this change, I believe it's better there with the other classes and src/wp-includes/block-supports
only includes the actual block supports not the framework for it.
63c6ee5
to
26df7a5
Compare
Ok this is looking good to me now, I'm going to proceed and commit this patch. |
…press-develop into update/npm-packages
Updates @WordPress npm packages to their latest versions, prior to 5.6 Beta 1.
Trac ticket: https://core.trac.wordpress.org/ticket/51570
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.