Skip to content
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

v1 fix: overwrite mem block timestamp if CKB median time is less than tip block #627

Merged
merged 1 commit into from
Mar 26, 2022
Merged

v1 fix: overwrite mem block timestamp if CKB median time is less than tip block #627

merged 1 commit into from
Mar 26, 2022

Conversation

jjyr
Copy link
Collaborator

@jjyr jjyr commented Mar 24, 2022 •
edited by Flouse
Loading

@jjyr jjyr requested review from Flouse and zeroqn March 24, 2022 02:06
@jjyr jjyr changed the title fix: overwrite mem block timestamp if CKB median time is less than tip block v1 fix: overwrite mem block timestamp if CKB median time is less than tip block Mar 24, 2022
@jjyr jjyr force-pushed the fix-timestamp-v1 branch from 0d841e1 to 8fb1f53 Compare March 24, 2022 03:52
Flouse
Flouse requested changes Mar 24, 2022
View reviewed changes
Copy link
Collaborator

@Flouse Flouse left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the CI error:
https://github.com/nervosnetwork/godwoken/runs/5670799787?check_suite_focus=true#step:7:386

@jjyr jjyr force-pushed the fix-timestamp-v1 branch from 8fb1f53 to 5cb4905 Compare March 24, 2022 07:02
@Flouse Flouse merged commit 5abb63f into godwokenrises:compatibility-breaking-changes Mar 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Flouse Flouse Flouse requested changes

@zeroqn zeroqn zeroqn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jjyr @Flouse @zeroqn