Skip to content

ci: add a PR title check and unify the title convention - #1980

Open
yeongseon wants to merge 1 commit into
mainfrom
ci/title-convention
Open

yeongseon wants to merge 1 commit into
mainfrom
ci/title-convention

Conversation

@yeongseon

Copy link
Copy Markdown
Owner

Closes #1979

What changes

  • CONTRIBUTING.md: new Title Convention section, the single source of truth
  • AGENTS.md, docs-site contributing page, PR template: point to it instead of restating rules
  • Issue forms: Conventional Commit default prefix where missing
  • .github/workflows/pr-title.yml: format + allowed-type check with an example on failure

Verification (local)

  • Title check logic: 24 cases (valid, scope, !, release-please and Dependabot titles, missing prefix, unknown/uppercase type, empty description, trailing period, quotes/backticks/$(...) injection attempts) all behave as expected
  • actionlint on every workflow; the repo's own linters, tests and strict docs build where applicable

This PR is not merged as part of this change. After it merges and the check runs green on main, register PR title as a required status check.

Adds a single Title Convention section to CONTRIBUTING.md and points AGENTS.md, the docs site, the issue forms and the PR template at it. Adds a minimal PR title check (pull_request event, no permissions, no checkout, title passed via env).
Copilot AI balanced review requested due to automatic review settings September 29, 2026 12:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The promised direct Title Convention link is missing from the documentation-site contributing page.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Centralizes title conventions and adds automated PR-title validation.

Changes:

  • Defines one canonical title format and allowed types.
  • Adds a secure, event-driven PR title check.
  • Updates templates with conventional prefixes and guidance.
File Description
CONTRIBUTING.md Defines the canonical title convention.
AGENTS.md References the canonical convention.
.github/​workflows/​pr-title.yml Validates PR titles.
.github/​pull_request_template.md Adds title guidance.
.github/​ISSUE_TEMPLATE/​user_story.yml Prefills feat:.
.github/​ISSUE_TEMPLATE/​question.yml Prefills docs:.
.github/​ISSUE_TEMPLATE/​feature_request.yml Prefills feat:.
.github/​ISSUE_TEMPLATE/​epic.yml Prefills feat:.
.github/​ISSUE_TEMPLATE/​bug_report.yml Prefills fix:.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CONTRIBUTING.md

## Title Convention

This section is the single source of truth for issue, pull request, and commit titles in this repository. `AGENTS.md`, the pull request template, the issue forms, and the documentation site point here instead of restating the rules.

This branch has not been deployed

No deployments
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.

ci: add a PR title check and unify the title convention

2 participants