docs: Refactor the What is Hiero section in _index.md#276
Merged
aceppaluni merged 2 commits intohiero-ledger:mainfrom Mar 14, 2026
Merged
docs: Refactor the What is Hiero section in _index.md#276aceppaluni merged 2 commits intohiero-ledger:mainfrom
aceppaluni merged 2 commits intohiero-ledger:mainfrom
Conversation
Signed-off-by: freya-docs <freya.docs.pp@gmail.com>
✅ Deploy Preview for hiero-open-source ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Contributor
📝 WalkthroughWalkthroughCondensed the six "What is Hiero?" homepage points (fair, fast, secure, decentralized, leaderless, open) into shorter, scannable statements; no structural changes to sections, icons, or exported/public signatures. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip You can get early access to new features in CodeRabbit.Enable the |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
content/_index.md (1)
247-247: Improve phrasing for technical accuracy.The phrase "secure from denial-of-service and centralization" is technically imprecise. Systems are typically described as "resistant to" attacks or "designed to prevent" centralization, not "secure from" them. Consider rephrasing for better technical accuracy.
✍️ Suggested improvement
-text = "Leader-based consensus is unfair and either slow or vulnerable to attacks. We believe in a fully leaderless protocol where every node participates equally. Our protocol is secure from denial-of-service and centralization." +text = "Leader-based consensus is unfair and either slow or vulnerable to attacks. We believe in a fully leaderless protocol where every node participates equally. Our protocol is resistant to denial-of-service attacks and prevents centralization."🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@content/_index.md` at line 247, Update the text string assigned to text to use technically accurate phrasing: replace "secure from denial-of-service and centralization" with wording like "resistant to denial-of-service attacks and designed to prevent centralization" (or "reduce centralization risk"), and ensure the full sentence reads clearly — e.g. keep the claim about a fully leaderless protocol where every node participates equally and then state it "improves resistance to denial-of-service attacks and reduces centralization risk."
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@content/_index.md`:
- Around line 227-237: The three [[section_what_is_hiero_points]] blocks
(identified by the headings "Hiero is <strong>fair</strong>", "Hiero is
<strong>fast</strong>", and "Hiero is <strong>secure</strong>") use inconsistent
capitalization of the term "hashgraph"/"Hashgraph"; update the text fields so
they consistently use lowercase "hashgraph" when referring to the algorithm
(e.g., change "Hashgraph is leaderless" to "hashgraph is leaderless") and ensure
all occurrences in these text properties match that convention.
---
Nitpick comments:
In `@content/_index.md`:
- Line 247: Update the text string assigned to text to use technically accurate
phrasing: replace "secure from denial-of-service and centralization" with
wording like "resistant to denial-of-service attacks and designed to prevent
centralization" (or "reduce centralization risk"), and ensure the full sentence
reads clearly — e.g. keep the claim about a fully leaderless protocol where
every node participates equally and then state it "improves resistance to
denial-of-service attacks and reduces centralization risk."
Signed-off-by: freya-docs <freya.docs.pp@gmail.com>
freya-docs
changed the title
Contributor
There was a problem hiding this comment.
♻️ Duplicate comments (1)
content/_index.md (1)
227-227:⚠️ Potential issue | 🟡 MinorInconsistent capitalization of "Hashgraph".
This line uses "Hashgraph" (capitalized), while lines 232 and 237 consistently use "hashgraph" (lowercase) when referring to the algorithm. For consistency, standardize to lowercase "hashgraph" throughout.
📝 Suggested fix for consistency
-text = "Public forums should be equally accessible. No one should manipulate your transactions to their advantage. Hashgraph is leaderless - toward a fairer world." +text = "Public forums should be equally accessible. No one should manipulate your transactions to their advantage. The hashgraph algorithm is leaderless - toward a fairer world."🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@content/_index.md` at line 227, Update the capitalization in the string assigned to text (the line containing text = "Public forums should be equally accessible. No one should manipulate your transactions to their advantage. Hashgraph is leaderless - toward a fairer world.") to use lowercase "hashgraph" so it matches other occurrences; edit the quoted sentence accordingly to replace "Hashgraph" with "hashgraph".
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Duplicate comments:
In `@content/_index.md`:
- Line 227: Update the capitalization in the string assigned to text (the line
containing text = "Public forums should be equally accessible. No one should
manipulate your transactions to their advantage. Hashgraph is leaderless -
toward a fairer world.") to use lowercase "hashgraph" so it matches other
occurrences; edit the quoted sentence accordingly to replace "Hashgraph" with
"hashgraph".
exploreriii
approved these changes
Mar 14, 2026
Contributor
exploreriii
left a comment
exploreriii
left a comment
There was a problem hiding this comment.
Hi i think you have done an amazing job at preserving the original meaning but writing in it less words, will pass to @aceppaluni for a second opinion
aceppaluni
approved these changes
Mar 14, 2026
Contributor
aceppaluni
left a comment
aceppaluni
left a comment
There was a problem hiding this comment.
@freya-docs Thank you so much for working on this. This looks great to me.
I will Merge!
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
4 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.
Description
This PR refactors descriptions in the What is Hiero section in _index.md
Related Issues
Closes: #263
Summary by CodeRabbit