Skip to content

fix(ci): enable conventional PR titles by default - #88

Merged
trakhimenok merged 1 commit into
mainfrom
ci/conventional-pr-title-default
Sep 12, 2026
Merged

trakhimenok merged 1 commit into
mainfrom
ci/conventional-pr-title-default

Conversation

@trakhimenok

Copy link
Copy Markdown
Member

Conventional PR-title validation currently requires each caller to opt in, so new workflow consumers can still miss releases because a squash title is unconstrained. Make the shared gate default to enabled while retaining require_conventional_pr_title: false as an explicit opt-out.

The documentation now shows the default and opt-out forms, and the workflow contract test locks the default to true.

Validation: go test ./..., go vet ./..., and git diff --check passed.

@trakhimenok

Copy link
Copy Markdown
Member Author

AI review: approved.

The diff changes only the reusable input default, its documentation, and the matching regression assertion. The conditional still honors an explicit false, so opt-out behavior remains intact. The PR's conventional title exercised the default-on path in the required Lint job, all seven exact-head checks passed, and go test ./..., go vet ./..., and git diff --check passed locally. I found no correctness or security issue.

@trakhimenok
trakhimenok merged commit 08dd785 into main Sep 12, 2026
7 checks passed
@trakhimenok
trakhimenok deleted the ci/conventional-pr-title-default branch September 12, 2026 08:05
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.

1 participant