-
Notifications
You must be signed in to change notification settings - Fork 35
feat: what is a node #1732
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
base: main
Are you sure you want to change the base?
feat: what is a node #1732
Conversation
|
/review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the clear new overview in what-is-a-node.mdx: I’ve left several suggestions there, so please apply the inline suggestions to align the wording and structure with the style guide.
| alt="Flow from a user wallet or dApp through an API or liteserver gateway to TON full nodes and validators." | ||
| /> | ||
|
|
||
| ## Run your own node (optional) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[HIGH] Non-imperative, descriptive heading on task section
The heading ## Run your own node (optional) combines a descriptive phrase (“your own node”) with an inline optionality label, instead of a concise imperative heading. The style guide requires task-oriented instructions to use imperative phrasing (“Run a node”) and to avoid extra qualifiers that reduce scanability. This heading is the entry point to an action-oriented section and should follow the same imperative, neutral style as steps and commands.
| ## Run your own node (optional) | |
| ## Run a node |
Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@novusnota @aigerimu wdyt?
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Closes #1683