Skip to content

Add Valkey support to test matrix#141

Merged
chrisguidry merged 1 commit intomainfrom
issue-139
Jun 20, 2025
Merged

Add Valkey support to test matrix#141
chrisguidry merged 1 commit intomainfrom
issue-139

Conversation

@chrisguidry
Copy link
Owner

@chrisguidry chrisguidry commented Jun 20, 2025

Summary

  • Adds Valkey 8.0 to the CI test matrix to verify compatibility alongside Redis 6.2 and 7.4
  • Updates test infrastructure to handle both Redis and Valkey Docker images
  • Maintains full backward compatibility with existing Redis testing

Changes

  • CI Matrix: Added valkey-8.0 to test matrix, creating 12 total test combinations
  • Test Infrastructure: Enhanced Docker image selection logic in conftest.py to support Valkey images
  • Documentation: Updated requirements to mention Valkey 8.0+ support

This closes #139 to test against Valkey as a Redis-compatible alternative.

🤖 Generated with Claude Code

- Add valkey-8.0 to CI test matrix alongside Redis 6.2 and 7.4
- Update test infrastructure to handle Valkey Docker images
- Maintain backward compatibility with existing Redis testing
- Update documentation to reflect Valkey support

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link

📚 Documentation has been built for this PR!

You can download the documentation directly here:
https://github.com/chrisguidry/docket/actions/runs/15768449997/artifacts/3366599612

@codecov-commenter
Copy link

codecov-commenter commented Jun 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (f62d09c) to head (d42ce21).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #141   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           27        27           
  Lines         3354      3358    +4     
  Branches       189       190    +1     
=========================================
+ Hits          3354      3358    +4     
Flag Coverage Δ
python-3.12 99.97% <75.00%> (-0.03%) ⬇️
python-3.13 99.97% <75.00%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
tests/conftest.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chrisguidry chrisguidry merged commit 79c4844 into main Jun 20, 2025
16 checks passed
@chrisguidry chrisguidry deleted the issue-139 branch June 20, 2025 00:24
chrisguidry added a commit that referenced this pull request Jun 20, 2025
## Summary
- Adds Valkey 8.0 to the CI test matrix to verify compatibility
alongside Redis 6.2 and 7.4
- Updates test infrastructure to handle both Redis and Valkey Docker
images
- Maintains full backward compatibility with existing Redis testing

## Changes
- **CI Matrix**: Added `valkey-8.0` to test matrix, creating 12 total
test combinations
- **Test Infrastructure**: Enhanced Docker image selection logic in
`conftest.py` to support Valkey images
- **Documentation**: Updated requirements to mention Valkey 8.0+ support

This closes #139 to test against Valkey as a Redis-compatible
alternative.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>
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.

Test against Valkey

2 participants