Skip to content

docs: Update README.md with grammatical fixes#4583

Merged
straker merged 1 commit intodequelabs:developfrom
HuanWuCode:develop
Sep 24, 2024
Merged

docs: Update README.md with grammatical fixes#4583
straker merged 1 commit intodequelabs:developfrom
HuanWuCode:develop

Conversation

@HuanWuCode
Copy link
Copy Markdown
Contributor

Update the doc for clarity and consistency with present tense. Also, "rules' description and help properties as well as each checks' metadata.messages property" → should be "rule's description and help properties, as well as each check's metadata.messages property" (singular possessive for "rule" and "check," and add a comma before "as well as").

Update the doc for clarity and consistency with present tense.
Also, "rules' description and help properties as well as each checks' metadata.messages property" → should be "rule's description and help properties, as well as each check's metadata.messages property" (singular possessive for "rule" and "check," and add a comma before "as well as").
@HuanWuCode HuanWuCode requested a review from a team as a code owner September 23, 2024 00:53
Copy link
Copy Markdown
Contributor

@straker straker left a comment

Choose a reason for hiding this comment

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

Thanks for the pr!

Reviewed for security

@straker straker changed the title Update README.md docs: Update README.md with grammatical fixes Sep 24, 2024
@straker straker merged commit b422c79 into dequelabs:develop Sep 24, 2024
straker pushed a commit that referenced this pull request Oct 16, 2024
Update the doc for clarity and consistency with present tense. Also,
"rules' description and help properties as well as each checks'
metadata.messages property" → should be "rule's description and help
properties, as well as each check's metadata.messages property"
(singular possessive for "rule" and "check," and add a comma before "as
well as").
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