Skip to content

docs: clarify required ruleset check names#158

Merged
charlesrhoward merged 1 commit intomainfrom
codex/issue-158-contributing-ruleset-checks
Feb 16, 2026
Merged

docs: clarify required ruleset check names#158
charlesrhoward merged 1 commit intomainfrom
codex/issue-158-contributing-ruleset-checks

Conversation

@charlesrhoward
Copy link
Copy Markdown
Contributor

@charlesrhoward charlesrhoward commented Feb 16, 2026

Summary

  • update contributor docs with the exact GitHub check names to require in rulesets
  • add pnpm test:coverage to the required local pre-PR command list

Validation

  • pnpm lint
  • pnpm typecheck

Note

Low Risk
Documentation-only change that adjusts contributor workflow guidance; no runtime or production code paths are affected.

Overview
Updates CONTRIBUTING.md to add pnpm test:coverage to the pre-PR required local commands.

Also documents the exact GitHub ruleset required check names (desktop/frontend build checks and smoke/coverage checks) to reduce ambiguity when configuring repository rulesets.

Written by Cursor Bugbot for commit f06aa54. This will update automatically on new commits. Configure here.

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agent-space-web Ready Ready Preview, Comment Feb 16, 2026 6:22pm

Request Review

@charlesrhoward charlesrhoward enabled auto-merge (squash) February 16, 2026 18:21
@charlesrhoward charlesrhoward merged commit 640d63d into main Feb 16, 2026
7 checks passed
@charlesrhoward charlesrhoward deleted the codex/issue-158-contributing-ruleset-checks branch February 16, 2026 18:23
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