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

node: use limiting proposer #157

Merged
merged 1 commit into from
Dec 31, 2023
Merged

node: use limiting proposer #157

merged 1 commit into from
Dec 31, 2023

Conversation

rphmeier
Copy link
Contributor

@rphmeier rphmeier commented Dec 28, 2023

This activates the limiting proposer from #156.

Copy link
Contributor Author

rphmeier commented Dec 28, 2023

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@rphmeier rphmeier marked this pull request as ready for review December 28, 2023 01:12
@rphmeier
Copy link
Contributor Author

@pepyakin you will be pleased to know I ran tests and demo rollups. The go rollup doesn't seem to work on my machine for unrelated reasons, but it is managing to communicate with the shim. Sovereign worked correctly and with the expected behavior.

@rphmeier
Copy link
Contributor Author

Also tested a runtime upgrade locally and it works as expected.

Copy link
Contributor

Thank you for taking the time!

Copy link
Contributor

pepyakin commented Dec 31, 2023

Merge activity

  • Dec 31, 7:58 AM: @pepyakin started a stack merge that includes this pull request via Graphite.
  • Dec 31, 7:59 AM: Graphite rebased this pull request as part of a merge.
  • Dec 31, 7:59 AM: @pepyakin merged this pull request with Graphite.

Base automatically changed from rh-limiting-proposer to main December 31, 2023 12:58
@pepyakin pepyakin merged commit ce24a42 into main Dec 31, 2023
4 checks passed
@pepyakin pepyakin deleted the rh-use-limiting-proposer branch December 31, 2023 12:59
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.

2 participants