-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Support COPY #2064
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
Support COPY #2064
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking good - thanks! There are a few things I'd like to remedy inline and test a bit better but overall great. I'm happy to make the suggested changes and push if preferred soon as time allows, let me know :)
Also adds a new version-based skip override so we can more simply apply this in ongoing PRs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Current is looking great - thanks for this!
https://redis.io/commands/copy/