Skip to content

test: add Vale auto-fix test page with intentional violations#574

Open
jth-nw wants to merge 1 commit intodevfrom
test/vale-autofix-v2
Open

test: add Vale auto-fix test page with intentional violations#574
jth-nw wants to merge 1 commit intodevfrom
test/vale-autofix-v2

Conversation

@jth-nw
Copy link
Collaborator

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

No description provided.

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

Documentation PR Review

Editorial Review

This file is explicitly a Vale auto-fix test page with intentional violations. The issues below are flagged for completeness — most are by design.

  • First person — Line 39: "We recommend that our users configure the monitoring plan before deployment." Uses prohibited first person ("We", "our"). Beyond the Vale violation, the construction is also indirect — documentation should give direct instructions rather than recommendations. Suggested fix: "Configure the monitoring plan before deployment."
  • Impersonal phrasing — Line 53: "The system allows you to configure multiple connections." Weak, indirect construction. Suggested fix: "You can configure multiple connections."
  • Temporal conjunction — Line 43: "Once the installation is complete, verify the service is running." Netwrix style prefers "After" over "Once" for time-based conditions. Suggested fix: "After the installation is complete, verify the service is running."

Summary

0 Dale issues, 3 editorial suggestions across 1 file.

Note: All issues in this file appear to be intentional — this is a test page for the Vale auto-fix workflow. No action is required unless validating that auto-fix targets the correct lines.


What to do next:
Comment @claude on this PR followed by your instructions. For example:

  • @claude fix all issues
  • @claude fix only the Dale issues
  • @claude reorganize the prerequisites section
  • @claude help improve the flow of this document

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

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