Skip to content

Fix/epp minor05052026#856

Merged
jth-nw merged 5 commits into
devfrom
fix/epp_minor05052026
May 5, 2026
Merged

Fix/epp minor05052026#856
jth-nw merged 5 commits into
devfrom
fix/epp_minor05052026

Conversation

@jth-nw
Copy link
Copy Markdown
Collaborator

@jth-nw jth-nw commented May 5, 2026

No description provided.

jth-nw and others added 2 commits May 5, 2026 10:28
…settings

Replaces Unicode ffi ligature (U+FB03) with ASCII in all 18 instances of
"traffic"/"Traffic", fixing two broken heading anchors in the process.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
Replaces fi, fl, ff, ffi, ffl ligatures (U+FB00–FB04) with ASCII
equivalents across 150 instances — fixes broken search, spell-check,
and heading anchors throughout the file.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
@jth-nw jth-nw requested a review from a team as a code owner May 5, 2026 15:33
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Auto-Fix Summary

9 issues fixed, 5 skipped across 1 files

Category Fixes
Contractions 2
Dale: minimizing-difficulty 1
Dale: passive-voice 6
Skipped (needs manual review) Reason
docs/endpointprotector/admin/dc_module/globalsettings.md:275 — Netwrix.OxfordComma False positive. 'enabled' is part of the conditional clause 'if enabled,' not a list item. 'network and browser traffic' is a two-item pair and does not require an Oxford comma.
docs/endpointprotector/admin/dc_module/globalsettings.md:417 — Netwrix.OxfordComma False positive in image alt text. Same pattern as line 275: 'enabled' is part of 'if enabled,' conditional clause, and 'network and browser' is only two items — no Oxford comma required.
docs/endpointprotector/admin/dc_module/globalsettings.md:395 — Dale: passive-voice Warning admonition starting with 'be aware that...manually checked within CAP policies' — meaning of 'checked' (checkbox selection) is ambiguous; rewrite could change intended meaning
docs/endpointprotector/admin/dc_module/globalsettings.md:617 — Dale: passive-voice 'files copied to MTP devices' is a noun phrase describing what is monitored, not a passive verb construction in this context
docs/endpointprotector/admin/dc_module/globalsettings.md:877 — Dale: passive-voice 'if Deep Packet Inspection is enabled' is a conditional clause; rewriting would disrupt the sentence flow without clear benefit

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

…e fix

Updates the link to globalsettings#dpi-configuration to use ASCII,
matching the heading anchor generated after the fi ligature was replaced.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
@jth-nw jth-nw temporarily deployed to development May 5, 2026 16:02 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Documentation PR Review

Editorial Review

docs/endpointprotector/admin/cap_module/deeppacket.md

No issues found.

docs/endpointprotector/admin/dc_module/globalsettings.md

No issues found.

Summary

0 editorial suggestions across 2 files. Both files received fi-ligature encoding fixes (fi → fi, ffi → ffi, etc.) throughout, along with a handful of active-voice rewrites (Memory and Disk Persistence, Clearing Bypass State, Metadata Scanning, Advanced Printer and MTP Scanning, shadow copy creation, Block Print from Browsers warning). All changes are clean. 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.

@jth-nw jth-nw merged commit fb27ac6 into dev May 5, 2026
9 of 10 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Auto-Fix Summary

42 issues fixed, 8 skipped across 2 files

Category Fixes
Dale: idioms 2
Dale: passive-voice 37
Dale: positional-references 2
Dale: undefined-acronyms 1
Skipped (needs manual review) Reason
docs/endpointprotector/admin/dc_module/globalsettings.md:275 — Netwrix.OxfordComma False positive: the pattern 'enabled, network and' contains only a two-item list ('network and browser traffic'). The comma before 'network' is a conditional clause separator ('if enabled, network and browser traffic...'), not a list comma. Oxford comma applies only to three or more items.
docs/endpointprotector/admin/dc_module/globalsettings.md:417 — Netwrix.OxfordComma False positive: same two-item pattern in image alt text ('If enabled, network and browser traffic can be inspected'). 'Network' and 'browser' are two items only; Oxford comma rule does not apply.
docs/endpointprotector/admin/cap_module/deeppacket.md:102 — Dale: undefined-acronyms Acronym 'UM' (likely User Manual) is undefined but its exact meaning is unclear from context; fix would risk changing intent
docs/endpointprotector/admin/cap_module/deeppacket.md:54 — Dale: passive-voice 'opting for Stealthy DPI can resolve these issues' is active voice; no fix needed
docs/endpointprotector/admin/dc_module/globalsettings.md:347 — Dale: passive-voice 'the DPI bypass is activated' — restructuring this conditional clause alongside surrounding technical context introduces ambiguity about the activating agent
docs/endpointprotector/admin/dc_module/globalsettings.md:600 — Dale: passive-voice 'that DLL is called' — the calling agent (the OS/runtime) is not well-defined in this context; passive is appropriate here
docs/endpointprotector/admin/dc_module/globalsettings.md:125 — Dale: passive-voice 'to no longer be logged' — restructuring this complex feature-description sentence would significantly alter its structure; ambiguous fix
docs/endpointprotector/admin/dc_module/globalsettings.md:386 — Dale: passive-voice 'are enabled' in the conditional clause 'If both... are enabled' — standard conditional state description; forcing active would be awkward

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