Skip to content

Conversation

@m4sterbunny
Copy link
Contributor

Description

  • Standardizes the syntax for True / False configs
  • Adds missing defaults

Issue(s) fixed

Partial fix for #1722

Preview

Signed-off-by: m4sterbunny <harrie.bickle-ext@consensys.net>
Signed-off-by: m4sterbunny <harrie.bickle-ext@consensys.net>
@m4sterbunny m4sterbunny requested a review from a team as a code owner November 28, 2025 11:25
@vercel
Copy link

vercel bot commented Nov 28, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
besu-docs Ready Ready Preview Comment Nov 28, 2025 11:27am

Copy link
Contributor

@alexandratran alexandratran left a comment

Choose a reason for hiding this comment

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

Minor comment, lgtm

Possible values include the following:

| Network | Chain | Type | Default sync mode | Consensus mechanism | Description |
| Network | Chain | Type | Default sync mode | Consensus mechanism | description |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| Network | Chain | Type | Default sync mode | Consensus mechanism | description |
| Network | Chain | Type | Default sync mode | Consensus mechanism | Description |

Capitalize for consistency

Copy link
Contributor

@macfarla macfarla left a comment

Choose a reason for hiding this comment

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

happy for this change.

I did notice at least one spot where we have a third way - but this could be addressed in a follow up PR

--era1-import-prepipeline-enabled[=<Boolean>]

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