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

Added note that min-block-occupancy-ratio is ignored for PoS #1354

Merged
merged 4 commits into from
Jul 5, 2023
Merged

Added note that min-block-occupancy-ratio is ignored for PoS #1354

merged 4 commits into from
Jul 5, 2023

Conversation

AnmolSirola
Copy link
Contributor

Added a note in the min-block-occupancy-ratio section that it is ignored for PoS.
Issue #1344

Signed-off-by: Anmol Sirola <95535448+AnmolSirola@users.noreply.github.com>
@AnmolSirola AnmolSirola requested a review from a team as a code owner July 1, 2023 13:24
@vercel
Copy link

vercel bot commented Jul 1, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
besu-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 5, 2023 8:18pm

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.

@AnmolSirola Thanks for contributing! Can you actually make these changes to the latest version of the docs, located in the docs/ folder, instead of the 23.4.1 version? This is to make sure your changes are included when we release the latest version.

Also, can you use the Docusaurus admonition syntax for notes?

@AnmolSirola
Copy link
Contributor Author

@alexandratran Sure, I will make changes in the latest version too. I will also use docusaurus syntax for adding the note.

Signed-off-by: Anmol Sirola <95535448+AnmolSirola@users.noreply.github.com>
@AnmolSirola
Copy link
Contributor Author

@alexandratran I have the required changes please have a look.

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.

@AnmolSirola I still only see these changes in the versioned_docs/version-23.4.1 folder, but as I mentioned in my previous comment, you need to make the changes in the docs/ folder instead. Can you update that, or would you prefer I edit your PR myself?

Signed-off-by: Anmol Sirola <95535448+AnmolSirola@users.noreply.github.com>
@AnmolSirola
Copy link
Contributor Author

@alexandratran I would like to apologize for my previous mistake. I misunderstood your instructions and implemented incorrect changes. However, I have now rectified the situation and made the necessary adjustments accordingly.

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.

Re-worded for simplicity, LGTM.

docs/public-networks/reference/cli/options.md Outdated Show resolved Hide resolved
Signed-off-by: Alexandra Tran Carrillo <12214231+alexandratran@users.noreply.github.com>
@alexandratran alexandratran merged commit 4833f80 into hyperledger:main Jul 5, 2023
10 of 11 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.

2 participants