-
Notifications
You must be signed in to change notification settings - Fork 225
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
fix updates on CLI side #3346
fix updates on CLI side #3346
Conversation
d30ab84
to
706a5e4
Compare
c402ea9
to
831886a
Compare
831886a
to
2f84f82
Compare
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.
lgtm
LGTM; we'll likely want that update-clonetask-in-place logic when we merge #3336, but it looks like it's easy enough to add back so not an issue :-) |
Are we planning to only support reclone-and-replay in the CLI? Or will we have some way of supporting both that and merge updates? |
This makes the CLI compatible with the changes in the backend from #3338. (First 3 commits here are the same as in #3338 but for some reason still show up after rebasing.)
Related: #3329