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

doc: update README.builder.md #14

Merged
merged 2 commits into from
Apr 2, 2024
Merged

doc: update README.builder.md #14

merged 2 commits into from
Apr 2, 2024

Conversation

pythonberg1997
Copy link
Collaborator

Description

This pr is to update README.builder.md

Rationale

tell us why we need these changes...

Example

add an example CLI or API response...

Changes

Notable changes:

  • update README.builder.md

- `Validators`: A list of validators to bid for.
- `Address`: The address of the validator.
- `URL`: The URL of the validator.
## FAQ

Choose a reason for hiding this comment

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

I think no need for this FAQ

## FAQ

1. Do builders fetch the in-turn proposer’s GasCeil to build block?
Yes, you could using RPC mev_params to query validator’s MEV information before building block, it can help to 1) calculate a valid header with gas no more than GasCeil; 2) calculate the left bidding time by BidSimulationLeftOver; 3) calculate suitable builderFee by validatorCommission.

Choose a reason for hiding this comment

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

can you merge the contend of https://github.com/bnb-chain/bsc-builder/blob/master/README.builder.md, and delete that file then?

@unclezoro unclezoro merged commit 0ca13b9 into develop Apr 2, 2024
5 of 6 checks passed
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