Skip to content

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

Open
jth-nw wants to merge 3 commits intodevfrom
test/vale-autofix-v4
Open

test: add Vale auto-fix test page with intentional violations#578
jth-nw wants to merge 3 commits intodevfrom
test/vale-autofix-v4

Conversation

@jth-nw
Copy link
Collaborator

@jth-nw jth-nw commented Mar 20, 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 20, 2026 00:05
@jth-nw jth-nw deployed to development March 20, 2026 00:06 — with GitHub Actions Active
@github-actions
Copy link
Contributor

Vale Auto-Fix Summary

9 issues fixed, 0 skipped across 1 files

Category Fixes
AllowsYouTo (rewrite) 1
BoilerplateCrossRef (rewrite) 1
FirstPersonPlural (rewrite) 1
HeadingPunctuation (rewrite) 3
NoteThat (rewrite) 1
OnceUsage (rewrite) 1
WeakLinkText (rewrite) 1

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

@github-actions
Copy link
Contributor

Documentation PR Review

Editorial Review

This file is explicitly a test page containing intentional style violations — all flagged patterns (passive voice, weak link text, first person, etc.) are there by design and labeled as such. No unintentional editorial issues found beyond the intentional violations.

One minor observation:

  • Clarity — Line 9: "It contains intentional violations." doesn't distinguish between Phase 1 and Phase 2 categories. A one-sentence note explaining the two-phase structure (e.g., "Phase 1 violations are fixed by script; Phase 2 violations are fixed by AI.") would help maintainers understand the test design at a glance.

Summary

0 Dale issues, 1 editorial suggestion across 1 file.


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