Skip to content

Fix duplicate word typos across docs#673

Open
jth-nw wants to merge 4 commits intodevfrom
fix/duplicate-word-typos
Open

Fix duplicate word typos across docs#673
jth-nw wants to merge 4 commits intodevfrom
fix/duplicate-word-typos

Conversation

@jth-nw
Copy link
Collaborator

@jth-nw jth-nw commented Mar 25, 2026

Remove repeated words ("is is", "and and", "of of", "as as", "for for") in PolicyPak, Access Analyzer, Endpoint Protector, and Auditor KB docs.

Remove repeated words ("is is", "and and", "of of", "as as", "for for")
in PolicyPak, Access Analyzer, Endpoint Protector, and Auditor KB docs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jth-nw jth-nw requested review from a team as code owners March 25, 2026 16:07
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Contributor

Documentation PR Review

Editorial Review

docs/accessanalyzer/12.0/solutions/filesystem/directpermissions/fs_nestedshares.md

No issues found.


docs/endpointprotector/install/virtualappliance/setupwizard.md

  • Clarity — Line 29: Residual duplicate-word typo "One one option must be selected." was not fixed alongside the "and and" → "and an" correction on the same line. Suggested fix: "Only one option must be selected."

  • Clarity — Line 29: "IPV4" and "IPV6" use non-standard capitalization on the modified line. The accepted abbreviations are "IPv4" and "IPv6" (lowercase "v"). Suggested fix: replace both instances — "an option to use IPv4 and an option for IPv6."


docs/policypak/components/scriptstriggers/manual/itemleveltargeting/overview.md

No issues found.


docs/policypak/components/startscreenandtaskbar/manual/collectionssettingsi/startscreen/overview.md

No issues found.


docs/policypak/gettingstarted/mdmmanual/xmldatafiles/taskbar.md

No issues found.


docs/policypak/gettingstarted/overviewinstall/overviewinstall.md

  • Clarity — Line 9: "Thank you for downloading Netwrix PolicyPak." is marketing language on the modified line. Netwrix writing standards require a neutral, informative tone with no marketing phrasing. Suggested fix: remove the sentence entirely, or replace with a neutral lead-in such as "This topic provides a quick overview of the installation process."

Summary

3 editorial suggestions across 2 files. Vale and Dale issues are auto-fixed separately.


What to do next:

Comment @claude on this PR followed by your instructions to get help:

  • @claude fix all issues — fix all editorial issues
  • @claude help improve the flow of this document — get writing assistance
  • @claude explain the voice issues — understand why something was flagged

You can ask Claude anything about the review or about Netwrix writing standards.

Automated fixes are only available for branches in this repository, not forks.

@github-actions
Copy link
Contributor

Auto-Fix Summary

31 issues fixed, 3 skipped across 6 files

Category Fixes
Contractions 6
Substitutions 5
AllowsYouTo (rewrite) 2
FirstPersonPlural (rewrite) 2
FollowTheStepsTo (rewrite) 1
NoteThat (rewrite) 1
Dale: idioms 2
Dale: passive-voice 7
Dale: positional-references 3
Dale: wordiness 2
Skipped (needs manual review) Reason

| docs/endpointprotector/install/virtualappliance/setupwizard.md:34 — Dale: passive-voice | Warning block contains a word-order typo ('creates certificate a regeneration') — meaning is unclear; fixing passive voice here risks further distorting the sentence |
| docs/accessanalyzer/12.0/solutions/filesystem/directpermissions/fs_nestedshares.md:47 — Dale: passive-voice | Table cell content 'where folders are exposed through multiple shares' — restructuring inside a table cell risks breaking the table or altering the report description meaning |
| docs/policypak/gettingstarted/overviewinstall/overviewinstall.md:9 — Dale: wordiness | 'Thank you for downloading Netwrix PolicyPak' — removing this would change the document's welcoming tone; no clearly better alternative without potentially altering intent |

Ask @claude on this PR if you'd like an explanation of any fix.

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