Skip to content

docs(README): Improve Documentation for better readability - #4672

Open
vinaypotluri wants to merge 2 commits into
block:mainfrom
vinaypotluri:vinaypotluri/improve-readme
Open

docs(README): Improve Documentation for better readability#4672
vinaypotluri wants to merge 2 commits into
block:mainfrom
vinaypotluri:vinaypotluri/improve-readme

Conversation

@vinaypotluri

@vinaypotluri vinaypotluri commented Aug 4, 2026

Copy link
Copy Markdown

Summary

Improve Documentation for better readability

Related issue

NA

Testing

Verified the Markdown renders correctly. Testing not required.

@vinaypotluri
vinaypotluri requested a review from a team as a code owner August 4, 2026 06:23
@vinaypotluri
vinaypotluri force-pushed the vinaypotluri/improve-readme branch from e966c81 to 2e460d8 Compare August 4, 2026 06:28
@vinaypotluri vinaypotluri changed the title README: Improve wording for better readability to avoid repeated phrasing docs(README): Improve wording for better readability to avoid repeated phrasing Aug 4, 2026
Signed-off-by: Vinay Potluri <vinaypotluri8@gmail.com>
@vinaypotluri
vinaypotluri force-pushed the vinaypotluri/improve-readme branch 2 times, most recently from 2e460d8 to 394cdb8 Compare August 4, 2026 08:35
@vinaypotluri vinaypotluri changed the title docs(README): Improve wording for better readability to avoid repeated phrasing docs(README): Improve Documentation for better readability Aug 4, 2026
@ravarora2 ravarora2 added the triage-ready Appropriate for agentic review label Aug 5, 2026
wpfleger96
wpfleger96 previously approved these changes Aug 5, 2026
@wpfleger96
wpfleger96 enabled auto-merge (squash) August 5, 2026 15:33

@wesbillman wesbillman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewing on Wes Pfleger's behalf.

Two changes are needed before this documentation cleanup is ready:

  1. NOSTR.md:57 adds - REVIEW to the protocol operation name (kind:9005 - REVIEW). That appears to be an internal editing marker, is unique in the Markdown corpus, and makes the reference table imply that REVIEW is part of the kind designation. Please remove it.

  2. docs/multi-tenant-relay.md:446-447 remains grammatically and semantically incomplete after replacing the with they: “together they fence the h-tag and claimed-community adversary cannot circumvent.” The second clause has no object, and it is unclear what is being fenced. Please rewrite the full clause rather than applying the one-word correction—for example, if this matches the intended model: “together they form the fence that the h-tag and claimed-community adversary cannot circumvent.”

The other three edits are clear grammatical/readability improvements. No test changes are involved.

Remove an accidental editing marker and clarify the multi-tenant
application-fence description.

Co-authored-by: Carl <c7ebe626f000404285d3686e1dc74cc07cc60a9754a150041ba132e14bd3e2ec@buzz.block.builderlab.xyz>
Signed-off-by: Wes <wesbillman@users.noreply.github.com>
@wesbillman
wesbillman dismissed their stale review August 5, 2026 16:15

Both requested documentation fixes were addressed in a2424f9; dismissing the stale changes-requested review. This is not an approval.

@vinaypotluri
vinaypotluri requested a review from wpfleger96 August 5, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

triage-ready Appropriate for agentic review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants