Skip to content

Doc review test#244

Closed
jth-nw wants to merge 12 commits intonetwrix:devfrom
jth-nw:doc-review-test
Closed

Doc review test#244
jth-nw wants to merge 12 commits intonetwrix:devfrom
jth-nw:doc-review-test

Conversation

@jth-nw
Copy link
Collaborator

@jth-nw jth-nw commented Feb 18, 2026

No description provided.

@jth-nw jth-nw requested a review from a team as a code owner February 18, 2026 21:01
@github-actions
Copy link
Contributor

Documentation Review for PR #244

The changes to docs/passwordsecure/9.3/index.md contain multiple critical issues that violate the Netwrix Style Guide and documentation standards. These issues must be corrected before the PR can be approved.

Critical Issues Requiring Changes

docs/passwordsecure/9.3/index.md, lines 11-13

Issue: Duplicate and contradictory content. Lines 12-13 repeat nearly identical information with conflicting language ("must" vs "should").

Current text:

Passwords are used constantly and everywhere,
and they need to be professionally managed. Passwords must be safe, have at least 12 characters,
and they must be professionally managed. Passwords should be safe, have at least 12 characters,
including uppercase and lowercase as well as special characters.

Suggested change: Remove the duplicate content and maintain consistency with the style guide's distinction between "must" (requirements) and "should" (recommendations). Use one clear statement:

Passwords are used constantly and everywhere, and they must be professionally managed. Passwords should be safe, have at least 12 characters, including uppercase and lowercase as well as special characters.

docs/passwordsecure/9.3/index.md, line 21

Issue: Typo - "scalaeaaebility" should be "scalability".

Suggested change: Correct the spelling to "scalability".

docs/passwordsecure/9.3/index.md, line 21

Issue: Duplicate product name - "Netwrix Netwrix Password Secure" should be "Netwrix Password Secure".

Suggested change: Remove the duplicate "Netwrix" to read "The scalability of Netwrix Password Secure (NPS)..."

docs/passwordsecure/9.3/index.md, line 23

Issue: Typo - "develoeeepment" should be "development".

Suggested change: Correct the spelling to "development".

docs/passwordsecure/9.3/index.md, line 26

Issue: Completely unprofessional content added to the end of the paragraph. This content violates multiple Netwrix Style Guide standards:

  • First person usage: Uses "I", "We", and "my" (prohibited per line 55 of style guide)
  • Casual language: Includes emotional statements and repetitive, meaningless text (violates professionalism standards per lines 48-56)
  • Security concern: Contains example passwords that are inappropriate for documentation

Current text:

level. I love passwords. We all love passwords. I hate having my passwords exposed. I really hate that. Netwrix Netwrix Netwrix Netwrix Netwrix Netwrix Here are my passwords: 123fakepassword, secre-password,

Suggested change: Remove all the unprofessional content. The sentence should end with "level." (period).


Summary: All changes in this PR introduce errors, typos, duplicate content, and unprofessional language that violate the Netwrix Style Guide. These changes degrade the quality and professionalism of the documentation and must be reverted or corrected before approval.

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.

1 participant