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.

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

feat(quint): refactor the blocksync spec. #545

Merged
merged 40 commits into from
Nov 15, 2024
Merged

feat(quint): refactor the blocksync spec. #545

merged 40 commits into from
Nov 15, 2024

Conversation

cason
Copy link
Contributor

@cason cason commented Nov 8, 2024

Part of #559.

Follow-up from #462


PR author checklist

Copy link
Contributor Author

Choose a reason for hiding this comment

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

General definitions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fast and can be tested in isolation.

Copy link

codecov bot commented Nov 8, 2024

The author of this PR, cason, is not an activated member of this organization on Codecov.
Please activate this user on Codecov to display this PR comment.
Coverage data is still being uploaded to Codecov.io for purposes of overall coverage calculations.
Please don't hesitate to email us at [email protected] with any questions.

@josef-widder josef-widder mentioned this pull request Nov 11, 2024
4 tasks
@josef-widder josef-widder mentioned this pull request Nov 11, 2024
2 tasks
Base automatically changed from josef/blocksync-retreat to main November 11, 2024 14:51
@josef-widder
Copy link
Member

There were some open points in #462 that we should fix here.

cason and others added 6 commits November 11, 2024 17:34
* Start Blocksync

* new Height enabled. needs to be tested

* blocksync fuctions done

* it is moving

* Daniel told me to push

* syncing on the lucky path

* syncing

* moved definitions a bit around

* code cleanup

* more cleanup

* polishing

* fix(quint): fix test units with new driver's methods and types (#524)

* quint: fix test unit with new driver's methods

* quint:  fix test units with new decision type

* quint: test-all.sh script with optional params

* Apply suggestions from code review

Co-authored-by: Josef Widder <[email protected]>

---------

Co-authored-by: Daniel <[email protected]>
@cason cason force-pushed the cason/462-blocksync branch from 28c45d7 to d03d5fd Compare November 11, 2024 16:36
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Replaced by blocksync directory.

@cason cason marked this pull request as ready for review November 11, 2024 17:45
@cason cason self-assigned this Nov 11, 2024
@cason cason added spec Related to specifications blocksync labels Nov 11, 2024
@cason cason mentioned this pull request Nov 14, 2024
7 tasks
Copy link
Member

@josef-widder josef-widder left a comment

Choose a reason for hiding this comment

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

Awesome refactor!! Let's merge it as this version builds a solid base to mock the consensus. (For mocking, everything except sync.qnt remains untouched.)

@cason cason merged commit aafc9f8 into main Nov 15, 2024
17 checks passed
@cason cason deleted the cason/462-blocksync branch November 15, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec Related to specifications
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants