-
Notifications
You must be signed in to change notification settings - Fork 499
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 #3662: Complete lightweight checkpointing #3686
Conversation
…nto move-hint-handler-to-domain
…o move-hint-handler-to-domain
@MaskedCarrot per my earlier comment, this isn't yet based on develop with passing CI. Please reassign at that point. |
Following on this, assign me back once this PR get develop branch as base branch. |
…resume-checkpoints
…htweight-checkpointing
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.
Code owner file LGTM
Unassigning @anandwana001 since they have already approved the PR. |
Assigning @vinitamurthi for code owner reviews. Thanks! |
I think I was added by mistake, I dont codeown any of these files. Removing myself! |
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.
Thanks @MaskedCarrot. LGTM!
Given everyone's approved, CI is passing, and there are not outstanding conversation threads, merging this. |
Explanation
Fixes #3662
Checklist
Final Result
lightweight_checkpointing.mp4
Espresso Tests
StateFragmentTest
ExplorationActivityTest