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 items in Readme Consensus #95

Merged
merged 8 commits into from
Dec 13, 2023
Merged

added items in Readme Consensus #95

merged 8 commits into from
Dec 13, 2023

Conversation

EdVeralli
Copy link
Contributor

Description of the Consensus Algortihm

@EdVeralli EdVeralli marked this pull request as ready for review December 10, 2023 23:57
README.md Show resolved Hide resolved
![](/img/consensus06.png)

The following selectSecureChain algorithm receives the peer's current best chain P.tip and its set of known valid chains P.chains and produces the most secure chain as output.
![](/img/consensus07.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why images instead of code blocks?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wanted to keep the colors in the functions and instructions. It's just that.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think that if you make a code block and set the proper language (Rust) it'll be colored. Here's a little example

Copy link
Contributor

Choose a reason for hiding this comment

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

However it's not a big deal right now, we can open an issue about this!

@xqft xqft merged commit 1c886a5 into main Dec 13, 2023
5 checks passed
@xqft xqft deleted the doc_consensus branch December 13, 2023 17:46
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