Skip to content

ci: lint pull request titles - #1170

Open
0xisk wants to merge 1 commit into
refactor-contributingfrom
ci/pr-title-lint
Open

0xisk wants to merge 1 commit into
refactor-contributingfrom
ci/pr-title-lint

Conversation

@0xisk

@0xisk 0xisk commented Oct 19, 2026

Copy link
Copy Markdown
Member

Stacked on #1139; until that merges this PR's diff includes its commits. Review only .github/workflows/pr-title.yml.

Types of changes

What types of changes does your code introduce to OpenZeppelin Midnight Contracts?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Closes #1169

Triggers on pull_request, not the pull_request_target the action's README recommends. The action only calls pulls.get, which the read-only token on fork PRs covers for a public repo, and the job never runs in the base repo's privileged context. Config changes also take effect in the PR that makes them.

PR Checklist

Further comments

After merge, a maintainer needs to add Lint PR Title (workflow PR Title) as a required check on the main ruleset. That is not part of this PR.

PRs are squash-merged with the PR title as the commit subject, so a
non-conforming title lands on main as written. #1139 documents the
Conventional Commits rule for titles, but no check enforces it.

Closes #1169
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 354ba87a-cace-4f29-ac48-886d80a05056

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@0xisk
0xisk changed the base branch from main to refactor-contributing October 19, 2026 11:05
@0xisk
0xisk marked this pull request as ready for review October 19, 2026 11:05
@0xisk
0xisk requested review from a team as code owners October 19, 2026 11:05

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: enforce Conventional Commits on PR titles

1 participant