Skip to content

ci: add PR title linting for conventional commits#150

Merged
nicknisi merged 1 commit into
mainfrom
nicknisi/lint-pr-title
May 13, 2026
Merged

ci: add PR title linting for conventional commits#150
nicknisi merged 1 commit into
mainfrom
nicknisi/lint-pr-title

Conversation

@nicknisi
Copy link
Copy Markdown
Member

@nicknisi nicknisi commented May 13, 2026

Summary

  • Adds lint-pr-title.yml GitHub Actions workflow to enforce conventional commit format on PR titles
  • Uses amannn/action-semantic-pull-request@v6 (pinned to SHA), matching the setup in authkit-nextjs
  • Runs on pull_request_target events (opened, edited, synchronize)

Test plan

  • Workflow file matches authkit-nextjs configuration
  • Verify the action runs on this PR (title uses ci: prefix)

Open in Devin Review

Summary by CodeRabbit

  • Chores
    • Added automated pull request title validation to maintain consistent commit message standards across the codebase.

Review Change Stack

Uses amannn/action-semantic-pull-request@v6 (pinned to SHA) to enforce
conventional commit format on PR titles, matching authkit-nextjs setup.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6ba04e99-94e7-4e0f-af37-ca1eddf3999f

📥 Commits

Reviewing files that changed from the base of the PR and between b2ad7bc and fa65766.

📒 Files selected for processing (1)
  • .github/workflows/lint-pr-title.yml

📝 Walkthrough

Walkthrough

This PR adds a new GitHub Actions workflow that validates pull request titles against semantic commit conventions. The workflow runs on PR open, edit, and synchronize events using the amannn/action-semantic-pull-request action with minimal permissions.

Changes

PR Title Lint Workflow

Layer / File(s) Summary
Semantic PR title validation workflow
.github/workflows/lint-pr-title.yml
New workflow enforces semantic PR title format on pull_request_target events (opened, edited, synchronize) using amannn/action-semantic-pull-request action with read-only pull-requests permission.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'ci: add PR title linting for conventional commits' directly and clearly describes the main change—adding a GitHub Actions workflow to enforce conventional commit formatting on PR titles.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch nicknisi/lint-pr-title

Comment @coderabbitai help to get the list of available commands and usage tips.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 13, 2026

Greptile Summary

This PR adds a new GitHub Actions workflow (.github/workflows/lint-pr-title.yml) to enforce the conventional commits format on PR titles using amannn/action-semantic-pull-request@v6, matching the setup in authkit-nextjs.

  • Triggers on pull_request_target with opened, edited, and synchronize event types; uses pull_request_target so the token is available even for fork-originated PRs.
  • The action is pinned to a full commit SHA (48f256284bd46cdaab1048c3721360e808335d50) for supply-chain safety, and permissions are scoped to pull-requests: read only — consistent with the minimal-permission pattern used in the existing CI workflow.

Confidence Score: 5/5

Safe to merge — adds a read-only linting workflow with no checkout step and minimal token permissions.

The workflow is a simple, self-contained addition. The action is SHA-pinned, the GITHUB_TOKEN is restricted to pull-requests: read, and no PR code is ever checked out or executed. No existing functionality is touched.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/lint-pr-title.yml New workflow to lint PR titles for conventional commits using amannn/action-semantic-pull-request@v6, pinned to SHA with minimal permissions.

Sequence Diagram

sequenceDiagram
    participant GitHub
    participant Workflow as lint-pr-title.yml
    participant Action as amannn/action-semantic-pull-request@v6

    GitHub->>Workflow: pull_request_target (opened/edited/synchronize)
    Workflow->>Action: Run with GITHUB_TOKEN (pull-requests: read)
    Action->>GitHub: Read PR title via event context
    alt Title matches conventional commit format
        Action-->>GitHub: ✅ Check passes
    else Title does not match
        Action-->>GitHub: ❌ Check fails
    end
Loading

Reviews (1): Last reviewed commit: "ci: add PR title linting for conventiona..." | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@nicknisi nicknisi merged commit 948a7e8 into main May 13, 2026
7 checks passed
@nicknisi nicknisi deleted the nicknisi/lint-pr-title branch May 13, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant