Closed ghost closed 3 months ago
Thanks for your interest in the rbuilder and being the first external contributor! Now about the code:
The PR seems to also change:
So it would be nice if you put that also on the description.
770b05a
Date (UTC) | 2024-07-05T07:05:27+00:00 |
Commit | 770b05ae295e308cbed634124a7c64e7ce9dfde7 |
Base SHA | a25f33b44d80a9ea12ac5ea2e68e54a89ce092a8 |
Benchmark | Mean | Status |
---|---|---|
MEV-Boost SubmitBlock serialization/JSON encoding | 10.74% | Performance has degraded. |
Don't mind the benchmark comment about degraded performance. It's a bit flaky, and should be fixed by https://github.com/flashbots/rbuilder/pull/15 which we intend to merge soon.
Don't mind the benchmark comment about degraded performance. It's a bit flaky, and should be fixed by #15 which we intend to merge soon.
Okay, I see.
Good work! Next time, try making separate PRs for things like lint/gitignore and "real" code changes. This time you get a free pass for being the first collaborator!
📝 Summary
.gitignore
💡 Motivation and Context
✅ I have completed the following steps:
make lint
make test