8000 Added test coverage for compatibility with Redis 8.0 by vladvildanov · Pull Request #1513 · predis/predis · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Added test coverage for compatibility with Redis 8.0 #1513

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

Merged
merged 30 commits into from
Feb 24, 2025

Conversation

vladvildanov
Copy link
Contributor
@vladvildanov vladvildanov commented Feb 24, 2025

Additional test coverage that tests new functions in Redis 8.0

@vladvildanov vladvildanov requested a review from a team February 24, 2025 13:30
@tillkruss tillkruss merged commit 02e2a7e into v2.x Feb 24, 2025
26 checks passed
@tillkruss tillkruss deleted the vv-8.0-compatibility branch February 24, 2025 17:22
vladvildanov added a commit that referenced this pull request Feb 25, 2025
* Added testing with 8.0

* Removed old service

* Moved checkout stage above image

* Changed service name

* Override config options for Redis < 7

* Exported to github env

* Print major version output

* Added double quotes

* Test fixes

* Merge cluster and standalone infrastructure

* Removed separate cluster job and merge together with standalone

* Merge stack tests into main test job

* Fixed annotation condition

* tweaks

* Moved docker-compose.yml to root folder

* Added compose file to expected

* fixed in v0.10.1

* move docker file

* Added -all profile in docker-compose

* Removed redis official image, reduce test matrix, moved docker-compose.yml

* Revert docker-compose moving

* Added test coverage to verify compatibility with 8.0

* Mark tests as relay-incompatible

* Codestyle fix

* Marked as relay-incompatible

---------

Co-authored-by: Till Krüss <tillkruss@users.noreply.github.com>
tillkruss pushed a commit that referenced this pull request Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0