-
Notifications
You must be signed in to change notification settings - Fork 68
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-771] Release 1.4.4 #450
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* replace default with main * update "rebase and merge" with merge commit this is still experimental but this way we don't bork our future selves by doing the wrong thing
* adds mVDP notes to readmes * Adds pr # to changelogs
…445) * use Pantheon-WP 2.0 * linting fixes * Don't error on mixed function & structure declaratiosn for object-cache.php * ignore specific sniffs that are not important * bump composer.lock * bump changelog * typo 🤦♂️ Co-authored-by: Phil Tyler <phil.tyler@pantheon.io> --------- Co-authored-by: Phil Tyler <phil.tyler@pantheon.io>
* add wpunit-helpers * bump php version for linting to 8.3 and add php execution check * add php8 compat checker * add test first pass still need to enable redis * bump composer.lock * add and use redis tests * update the phpcs:ignore line * maybe start redis? * setup and test redis * test with redis disabled * check if the database exists before running redis-enabled tests * fine.. * remove phpunit tests from circle * add some linebreaks between the phpunit runs * add -e to echoes * add emoji for the tests so they're easier to see and because I'm a dork * remove scripts sourced from wpunit-helpers from source control * add a matrix for redis enabled true/false * use a single script and export the WP_REDIS_USE_CACHE_GROUPS var for redis tests * we don't need the redis test now * we don't need the line breaks anymore
* bump tested up to version * echo the php version so we can ensure that we're testing on 8.3 * use the right terminus command
* fix: Handle duplicate keys in `get_multiple` function Signed-off-by: Souptik Datta <souptikdatta2001@gmail.com> * add readme update * fix whitespace 🤦♂️ --------- Signed-off-by: Souptik Datta <souptikdatta2001@gmail.com> Co-authored-by: Chris Reynolds <chris.reynolds@pantheon.io> Co-authored-by: Chris Reynolds <chris@jazzsequence.com>
jazzsequence
force-pushed
the
release_1.4.4
branch
from
November 27, 2023 17:44
317d9d0
to
e17e7a5
Compare
pwtyler
approved these changes
Nov 27, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
get_multiple
function [#448] (props @Souptik2001)