Repository navigation
Typos check - #47
Merged
Merged
Typos check#47
Conversation
arminsabouri
force-pushed
the
typos-check
branch
from
October 7, 2026 03:49
2e0c5d8 to
efa817b
Compare
bc1cindy
approved these changes
Oct 7, 2026
| Overcoming the community analysis concern requires a Sybil resistant, permissionless protocol where peers with no meaningful economic relationship can find each other. This involves an additional *coalition formation* protocol that sets up transaction construction using a market based mechanism. | ||
|
|
||
| While it is beneficial for privacy, interacting with strangers in a permissionless protocol invites distruption or attempts are survaillance. For honest peers to be able reliably transact, especially with the unanimity requirement, demands a byzantine fault tolerance from the transaction construction protocol. | ||
| While it is beneficial for privacy, interacting with strangers in a permissionless protocol invites distruption or attempts are surveillance. For honest peers to be able reliably transact, especially with the unanimity requirement, demands a byzantine fault tolerance from the transaction construction protocol. |
Member
There was a problem hiding this comment.
nit: three more on this line
("distruption", "attempts are", "to be able reliably").
Suggested change
| While it is beneficial for privacy, interacting with strangers in a permissionless protocol invites distruption or attempts are surveillance. For honest peers to be able reliably transact, especially with the unanimity requirement, demands a byzantine fault tolerance from the transaction construction protocol. | |
| While it is beneficial for privacy, interacting with strangers in a permissionless protocol invites disruption or attempts at surveillance. For honest peers to be able to reliably transact, especially with the unanimity requirement, demands a byzantine fault tolerance from the transaction construction protocol. |
Collaborator
Author
There was a problem hiding this comment.
How did it miss "distruption"?! We may need a better tool
Member
There was a problem hiding this comment.
I think the best tool for this are LLMs
I'll set the @oakagent to check typos and if needed post all in one comment
Collaborator
Author
There was a problem hiding this comment.
I dont know if we need to burn tokens for this. reliable spell checkers existed long before LLMs.
yuval-block
approved these changes
Oct 7, 2026
yuval-block
left a comment
yuval-block
left a comment
Collaborator
There was a problem hiding this comment.
concept ACK, code ACK it just doesn't close the reference issue, crate-ci/typos is much narrower in scope, but seems independently useful.
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #46