Skip to content

ci: standardize the PR title check (FB-4904) - #552

Merged
fstr merged 1 commit into
mainfrom
ci/FB-4904-pr-title-check
Oct 16, 2026
Merged

fstr merged 1 commit into
mainfrom
ci/FB-4904-pr-title-check

Conversation

@fstr

@fstr fstr commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Adds .github/workflows/pr-title.yml with the job check-title, using firebolt-db/action-pr-checks v2.0.0. The title must be a conventional commit (build, chore, ci, docs, feat, fix, perf, refactor, revert, style, test) whose subject doesn't start with an uppercase character. A ticket reference such as (FB-123) is optional. Bot PRs are skipped.
  • Removes the previous title check: .github/workflows/check-pr-formatting.yml.
  • check-title is the same in every public repo and becomes a required status check.

Part of FB-4904.

🤖 Generated with Claude Code


Note

Low Risk
Only GitHub Actions workflow definitions change; no application or runtime behavior is affected.

Overview
Replaces the repo-specific PR title workflow with the shared PR title check used across public repos.

The removed check-pr-formatting job (amannn/action-semantic-pull-request on main / 0.x, explicit Dependabot skip, lowercase-subject pattern) is superseded by check-title in pr-title.yml, which runs firebolt-db/action-pr-checks v2.0.0 on all pull requests (opened, edited, reopened, synchronize) and adds workflow concurrency so in-flight runs cancel when the PR updates.

This aligns required status check naming and rules (conventional commit type, subject not starting with uppercase, optional ticket ref) with the org standard from FB-4904.

Reviewed by Cursor Bugbot for commit 4a2ee09. Bugbot is set up for automated code reviews on this repo. Configure here.

Add .github/workflows/pr-title.yml with the job check-title, using
firebolt-db/action-pr-checks v2.0.0: the title must be a conventional
commit whose subject doesn't start with an uppercase character. A ticket
reference is optional. Bot PRs are skipped. Replaces the previous title
check, if any.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@fstr
fstr requested a review from a team as a code owner October 16, 2026 15:59
@sonarqubecloud

Copy link
Copy Markdown

@fstr
fstr merged commit a398aa7 into main Oct 16, 2026
17 of 18 checks passed
@fstr
fstr deleted the ci/FB-4904-pr-title-check branch October 16, 2026 16:49
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.

2 participants