Skip to content

Conversation

@keroro520
Copy link

@keroro520 keroro520 commented Jun 7, 2023

Mirror of #29, for reviewers to know what changes we made compared to the original fork repository.

}

func (s *l2VerifierBackend) SequencerStopped(ctx context.Context) bool {
return true
Copy link

Choose a reason for hiding this comment

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

why return true?

Copy link
Author

Choose a reason for hiding this comment

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

This module is for integration tests only. And l2VerifierBackend.SequencerStopped() will never be used. I just implement it because of Go interface requirement.


func (c *CoordinatorClient) RequestBuildingBlock() bool {
var respErr error
err := c.rpc.Call(respErr, "coordinator_requestBuildingBlock", c.sequencerId)
Copy link

Choose a reason for hiding this comment

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

should be a pointer?

Copy link
Author

Choose a reason for hiding this comment

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

I don't have a clue. It just works...

2129zxl pushed a commit that referenced this pull request Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants