Skip to content

implement XCLAIM command #1557

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
Merged
vladvildanov merged 6 commits into from
Jun 13, 2025
Merged

implement XCLAIM command #1557

vladvildanov merged 6 commits into from
Jun 13, 2025

Conversation

@alschastny
Copy link
Contributor

@alschastny alschastny commented Jun 11, 2025 •
edited
Loading

Added implementation of the XCLAIM command.

A more complex command tests can be performed after implementing the XPENDING command.

@alschastny alschastny requested a review from tillkruss as a code owner June 11, 2025 20:50
@tillkruss tillkruss requested a review from vladvildanov June 11, 2025 21:48
@coveralls
Copy link

coveralls commented Jun 12, 2025 •
edited
Loading

Coverage Status

coverage: 92.759% (+0.03%) from 92.729%
when pulling 36599ed on alschastny:xclaim-command
into b5b486a on predis:main.

vladvildanov
vladvildanov requested changes Jun 12, 2025
View reviewed changes
@alschastny alschastny requested a review from vladvildanov June 12, 2025 11:38
vladvildanov
vladvildanov requested changes Jun 12, 2025
View reviewed changes
@vladvildanov vladvildanov merged commit 0a8fe54 into predis:main Jun 13, 2025
41 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tillkruss tillkruss tillkruss approved these changes

@vladvildanov vladvildanov vladvildanov approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@alschastny @coveralls @tillkruss @vladvildanov