Skip to content
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 10 commits into from
Nov 27, 2023
Merged

[CMSP-771] Release 1.4.4 #450

merged 10 commits into from
Nov 27, 2023

Conversation

jazzsequence
Copy link
Contributor

@jazzsequence jazzsequence commented Nov 27, 2023

  • Updates Pantheon WP Coding Standards to 2.0 [#445]
  • Handle duplicate keys in get_multiple function [#448] (props @Souptik2001)

jazzsequence and others added 10 commits June 26, 2023 11:18
* 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 jazzsequence requested review from a team as code owners November 27, 2023 16:09
@jazzsequence jazzsequence changed the base branch from default to release November 27, 2023 17:41
@scottbuscemi scottbuscemi changed the title Release 1.4.4 [CMSP-771] Release 1.4.4 Nov 27, 2023
@jazzsequence jazzsequence merged commit ec6ddd9 into release Nov 27, 2023
20 checks passed
@jazzsequence jazzsequence deleted the release_1.4.4 branch November 27, 2023 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants