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

Validate builder configs in CI #294

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

0xmichalis
Copy link
Contributor

πŸ“ Summary

Add a CLI tool to validate the builder config and run it in CI to validate all configs found in the root of the repo.

πŸ’‘ Motivation and Context

Fix #278


βœ… I have completed the following steps:

  • Run make lint
  • Run make test
  • Added tests (if applicable)

Copy link

github-actions bot commented Dec 18, 2024

Benchmark results for a235b79

Report: https://flashbots-rbuilder-ci-stats.s3.us-east-2.amazonaws.com/benchmark/a235b79-2e4c1ab/report/index.html

Date (UTC) 2024-12-18T16:42:35+00:00
Commit a235b79994fe9264a83b1a360e244503271ce9e0
Base SHA 2e4c1ab8bc46e6110ef58fe39ac46160f6a7abbe

Significant changes

Benchmark Mean Status
gather_nodes_shared_cache 6.93% Performance has degraded.
MEV-Boost SubmitBlock serialization/JSON encoding -37.72% Performance has improved.

@ferranbt ferranbt merged commit 8989fca into flashbots:develop Dec 18, 2024
6 checks passed
@0xmichalis 0xmichalis deleted the validate-configs branch December 18, 2024 17:38
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.

CI step to validate that the README config files are valid
2 participants