Skip to content

Cleanup for Intel/AMD x64 vs. ARM architecture support on Linux - #1508

Open
DanPiazza-Netwrix wants to merge 1 commit into
devfrom
DanPiazza-Netwrix/matrix-intel-amd-arm-cleanup
Open

Cleanup for Intel/AMD x64 vs. ARM architecture support on Linux#1508
DanPiazza-Netwrix wants to merge 1 commit into
devfrom
DanPiazza-Netwrix/matrix-intel-amd-arm-cleanup

Conversation

@DanPiazza-Netwrix

Copy link
Copy Markdown
Contributor

Cleanup for Intel/AMD x64 vs. ARM architecture support on Linux.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Documentation PR Review

Editorial Review

docs/changetracker/requirements/supportmatrix.md

  • Completeness — Lines 44, 58, and 68: Removing "This table assumes Intel/AMD x64 architecture." leaves the reader without a stated default architecture for the Linux support tables. The tables list operating systems but no architecture column, so a reader on ARM (or any non-x64 platform) has no way to tell from this page whether the ✓ marks apply to them. If the removal is intentional because x64 is no longer the only assumed architecture, the page should say what architectures the tables now cover rather than dropping the statement entirely. Suggested fix: replace with a sentence that states the current baseline — for example, "These tables cover Intel/AMD x64 and ARM64 architectures." — or, if the baseline genuinely varies by row, add an Architecture column to the tables.

  • Clarity — Lines 44, 58, and 68: With the preceding sentence gone, "Express Agent is also available on 32 bit and s390x architectures upon request." now opens with a comparison that has no referent. The reader is told Express Agent is available on additional architectures without being told which architectures are the standard ones. Suggested fix: either restore a baseline sentence before it, or drop the comparative word: "Express Agent is available on 32 bit and s390x architectures upon request."

  • Clarity — Lines 44, 58, and 68: Since these lines were touched, worth correcting the compound modifier: "32 bit" should be hyphenated when it modifies "architectures." Suggested fix: "32-bit and s390x architectures".

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

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Auto-Fix Summary

0 issues fixed, 0 skipped across 1 files

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.

2 participants