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

Three suggestions for this text: Update node-configuration.mdx #3361

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Akare123
Copy link

Hi,

I have three suggestions to improve this text gramatically.

  1. "Several options are available here, ensure you've selected the one that best fits your nodes storage capabilities."

Missing apostrophe in "node's." Suggested fix: "Several options are available here; ensure you've selected the one that best fits your node's storage capabilities."

  1. "If you've configured a domain name for your node, this is the place to inform your node of it."

The phrase is awkward. Suggested fix: "If you've configured a domain name for your node, specify it here."

  1. "Seed mode, in which node constantly crawls the network and looks for peers."

Missing article. Suggested fix: "Seed mode, in which the node constantly crawls the network and looks for peers."

Thanks.

Hi,

I have three suggestions to improve this text gramatically.

1. "Several options are available here, ensure you've selected the one that best fits your nodes storage capabilities."

Missing apostrophe in "node's." Suggested fix: "Several options are available here; ensure you've selected the one that best fits your node's storage capabilities."

2. "If you've configured a domain name for your node, this is the place to inform your node of it."

The phrase is awkward. Suggested fix: "If you've configured a domain name for your node, specify it here."

3. "Seed mode, in which node constantly crawls the network and looks for peers."

Missing article. Suggested fix: "Seed mode, in which the node constantly crawls the network and looks for peers."

Thanks.
Copy link

vercel bot commented Dec 11, 2024

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
site ⬜️ Ignored (Inspect) Visit Preview Dec 11, 2024 2:40am

Copy link

vercel bot commented Dec 11, 2024

@Akare123 is attempting to deploy a commit to the unionbuild Team on Vercel.

A member of the Team first needs to authorize it.

@aeryz
Copy link
Contributor

aeryz commented Dec 20, 2024

Hey @Akare123! Thanks for the PR! It looks good but you need to modify your commit message for CI to succeed. See our contribution guides.

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