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.

Sign up for GitHub

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

Jump to bottom

Support for redis/v9 #697

Merged
merged 3 commits into from
Feb 9, 2023
Merged

Support for redis/v9 #697

merged 3 commits into from
Feb 9, 2023

Conversation

gaby
Copy link
Member

@gaby gaby commented Jan 30, 2023 •
edited
Loading

  • Dependency go-redis v9 was released today: https://github.com/redis/go-redis/releases
  • Added support for running tests against Redis v6.x and v7.x
  • Deprecated support for go 1.16 for the Redis storage driver
  • This PR updates the version from v8 to v9, and it also fixes the import path.

@gaby gaby requested a review from ReneWerner87 January 30, 2023 14:37
@gaby
Copy link
Member Author

gaby commented Jan 30, 2023

Looks like go-redis no longer supports go 1.16.

@gaby
Update Redis to v9. Fix import path
56599be 
Signed-off-by: Juan Calderon-Perez <[email protected]>
@gaby
Copy link
Member Author

gaby commented Jan 31, 2023

@ReneWerner87 Thoughts on bumping the Redis storage to go 1.17 or 1.18?

@gaby gaby changed the title Update Redis to v9 Support for redis/v9 Feb 8, 2023
@gaby
Copy link
Member Author

gaby commented Feb 9, 2023

@ReneWerner87 This should be ready for merge.

@ReneWerner87 ReneWerner87 merged commit d36b6ea into main Feb 9, 2023
@gaby gaby deleted the redis-v9 branch February 10, 2023 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ReneWerner87 ReneWerner87 ReneWerner87 approved these changes

@efectn efectn efectn approved these changes

Assignees
No one assigned
Labels
🧹 Updates
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gaby @ReneWerner87 @efectn