Skip to content

Make docs-review advisory-only#115

Merged
theletterf merged 1 commit intomainfrom
fab/docs-review-comment-only-2
Apr 27, 2026
Merged

Make docs-review advisory-only#115
theletterf merged 1 commit intomainfrom
fab/docs-review-comment-only-2

Conversation

@theletterf
Copy link
Copy Markdown
Member

Summary

  • make the reusable docs-review workflow submit comment-only reviews instead of request-changes reviews
  • remove low-signal review summary boilerplate from the review template
  • document that only docs-check-style mentions Vale and that the workflow currently falls back to manual style review when Vale is unavailable

Test plan

  • Run make compile.
  • Confirm compilation succeeds with 0 error(s), 0 warning(s).
  • Run the workflow on a test PR and verify it leaves only a non-blocking review comment.

Made with Cursor

Limit docs-review to non-blocking comment reviews, remove low-signal review-summary boilerplate, and document the current Vale dependency behavior in the workflow README.

Co-authored-by: GPT-5.4 <gpt-5.4@openai.com>
Made-with: Cursor
@theletterf theletterf requested a review from a team as a code owner April 27, 2026 08:36
@theletterf theletterf requested a review from cotti April 27, 2026 08:36
@theletterf theletterf self-assigned this Apr 27, 2026
@theletterf theletterf added the fix label Apr 27, 2026
@theletterf theletterf merged commit 7db01ad into main Apr 27, 2026
4 checks passed
@theletterf theletterf deleted the fab/docs-review-comment-only-2 branch April 27, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant