Skip to content

ci: pr conventional commits validator#260

Merged
SoulPancake merged 1 commit intomainfrom
ci/pr-cc-val
Mar 16, 2026
Merged

ci: pr conventional commits validator#260
SoulPancake merged 1 commit intomainfrom
ci/pr-cc-val

Conversation

@SoulPancake
Copy link
Member

@SoulPancake SoulPancake commented Mar 16, 2026

Description

What problem is being solved?

How is it being solved?

What changes are made to solve it?

References

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflow configuration to enforce pull request title conventions during the review process.

@SoulPancake SoulPancake requested a review from a team as a code owner March 16, 2026 08:06
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 16, 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: 049c571f-0105-46d5-8c0c-586a267046e2

📥 Commits

Reviewing files that changed from the base of the PR and between 4f89e71 and a447f8b.

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

Walkthrough

A new GitHub Actions workflow is added that validates PR titles conform to conventional commit conventions. The workflow triggers on PR events and runs the pr-conventional-commits action with specific allowed commit types and configuration options.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/pr-title-conventional-commit.yml
New workflow file that enforces conventional commit format for PR titles using the ytanikin/pr-conventional-commits action (v1.5.1), triggered on PR open/reopen/synchronize/edit events targeting the main branch. Configured with allowed task types (feat, fix, docs, test, refactor, ci, perf, chore, revert) and label creation disabled.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: adding a CI workflow for PR title validation using conventional commits conventions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ci/pr-cc-val
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@codecov-commenter
Copy link

codecov-commenter commented Mar 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.17%. Comparing base (4f89e71) to head (a447f8b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #260   +/-   ##
=======================================
  Coverage   71.17%   71.17%           
=======================================
  Files         137      137           
  Lines       11100    11100           
=======================================
  Hits         7900     7900           
  Misses       3200     3200           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SoulPancake SoulPancake added this pull request to the merge queue Mar 16, 2026
Merged via the queue into main with commit ba2fda2 Mar 16, 2026
30 checks passed
@SoulPancake SoulPancake deleted the ci/pr-cc-val branch March 16, 2026 17:59
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.

3 participants