-
Notifications
You must be signed in to change notification settings - Fork 62
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
[CMSP-1034] fix typo in tax_query #622
Conversation
Behat tests that are failing are on multidev deletion, the actual tests are passing. I've modified the |
@@ -1,6 +1,7 @@ | |||
## Changelog ## | |||
|
|||
### 2.5.3-dev ### | |||
### 2.5.3 (April 24, 2024) ### |
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 should happen in release PR, not feature branch.
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 branched off main and then went back and changed things when I realized my mistake. See ee583cf
@@ -235,7 +235,8 @@ Please report security bugs found in the source code of the Solr Power plugin th | |||
|
|||
== Changelog == | |||
|
|||
= 2.5.3-dev = | |||
= 2.5.3 (April 24, 2024) = |
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.
as above this should be release pr but fine if you're shipping this manually.
* Prepare 2.5.3-dev * [JFDI] Plugin deploy uses checkout@v3 (#614) * [CMSP-721] Bump tested-up-to and output PHP Version (#617) * bump tested-up-to * output the PHP version so we can ensure we're testing on PHP 8.3 * Bump ip from 2.0.0 to 2.0.1 (#619) Bumps [ip](https://github.com/indutny/node-ip) from 2.0.0 to 2.0.1. - [Commits](indutny/node-ip@v2.0.0...v2.0.1) --- updated-dependencies: - dependency-name: ip dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump yoast/phpunit-polyfills from 2.0.0 to 2.0.1 (#621) Bumps [yoast/phpunit-polyfills](https://github.com/Yoast/PHPUnit-Polyfills) from 2.0.0 to 2.0.1. - [Release notes](https://github.com/Yoast/PHPUnit-Polyfills/releases) - [Changelog](https://github.com/Yoast/PHPUnit-Polyfills/blob/2.x/CHANGELOG.md) - [Commits](Yoast/PHPUnit-Polyfills@2.0.0...2.0.1) --- updated-dependencies: - dependency-name: yoast/phpunit-polyfills dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [CMSP-1034] fix typo in tax_query (#622) * close parenthesis * add debugging code * add opening parenthesis Co-authored-by: Phil Tyler <[email protected]> * make the emoji sad * remove debug code * update changelog * bump version * if multidev:delete fails, don't fail the test --------- Co-authored-by: Phil Tyler <[email protected]> * drop -dev --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Phil Tyler <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Adds customer-supplied code for tax_queries from https://wordpress.org/support/topic/issue-building-query-when-you-are-parsing-the-tax_query/ props @offshorealert,