Skip to content

fix(docs): correct alt text for msi package image in EPP Intune deplo… - #1226

Open
brandonwest-netwrix wants to merge 4 commits into
devfrom
brandonwest-netwrix-EPP-Fix-Image
Open

fix(docs): correct alt text for msi package image in EPP Intune deplo…#1226
brandonwest-netwrix wants to merge 4 commits into
devfrom
brandonwest-netwrix-EPP-Fix-Image

Conversation

@brandonwest-netwrix

Copy link
Copy Markdown
Collaborator

fix(docs): correct alt text for msi package image in EPP Intune deployment guide
Generated with AI

Co-Authored-By: Claude Code ai@netwrix.com

…yment guide

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
@github-actions

Copy link
Copy Markdown
Contributor

Auto-Fix Summary

1 issues fixed, 1 skipped across 1 files

Category Fixes
Dale: passive-voice 1
Skipped (needs manual review) Reason

| docs/endpointprotector/install/intune/windowsdeployment.md:24 — Dale: passive-voice | Same construction appears in image alt text, which is descriptive metadata intentionally mirroring the warning callout, not body prose. |

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

@brandonwest-netwrix brandonwest-netwrix left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@netwrix/endpointprotector-docs
this check corrects the display of an image on windowsdeployment.md
No other file is changed

@brandonwest-netwrix brandonwest-netwrix added the endpoint-protector This change or issue involves Endpoint Policy Protector. label Jul 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Documentation PR Review

Editorial Review

docs/endpointprotector/install/intune/windowsdeployment.md

  • Clarity — Line 24: The new image alt text is a verbatim copy of the warning instruction ("When deploying the .msi package, delete the information contained in the brackets as well as the underscore that precedes it - EPPClientSetup.5.6.3.1_x86_64.msi"). Alt text should describe what the image shows so a reader using a screen reader understands the visual, not repeat the adjacent warning. Suggested fix: describe the image content, e.g. "The MSI package filename with the bracketed build information and preceding underscore highlighted for deletion".
  • Consistency — Line 24: The alt text says "information contained in the brackets," but the warning on line 19 was just edited to "information in the brackets." The two now disagree. Suggested fix: align the wording — drop "contained" from the alt text (or, better, rewrite the alt text as a description per the note above).
  • Clarity — Line 19: The warning uses a hyphen to introduce the example filename ("precedes it - EPPClientSetup..."). A hyphen reads as a compound-word connector here; an em dash or a colon would signal the example more clearly. Suggested fix: "delete the information in the brackets as well as the underscore that precedes it — for example, EPPClientSetup.5.6.3.1_x86_64.msi becomes EPPClientSetup.msi." This also clarifies what the file should look like after the edit.

Summary

3 editorial suggestions across 1 file. 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
Copy Markdown
Contributor

Auto-Fix Summary

1 issues fixed, 1 skipped across 1 files

Category Fixes
Dale: passive-voice 1
Skipped (needs manual review) Reason

| docs/endpointprotector/install/intune/windowsdeployment.md:9 — Dale: undefined-acronyms | MSI (Microsoft Installer package) is a well-known IT/Windows term for the sysadmin audience; rule guidance says not to flag well-known standards. |

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

endpoint-protector This change or issue involves Endpoint Policy Protector.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants