Skip to content

ci: fix conventional commit check#5744

Merged
jouho merged 2 commits intoaws:mainfrom
jouho:cc
Feb 17, 2026
Merged

ci: fix conventional commit check#5744
jouho merged 2 commits intoaws:mainfrom
jouho:cc

Conversation

@jouho
Copy link
Copy Markdown
Contributor

@jouho jouho commented Feb 17, 2026

Goal

Fix the PR title conventional commit validation so it actually runs on pull requests.

Why

The validate-pr-title job's if condition checks for pull_request_target, but the workflow is triggered by pull_request. Since these are different event types, the validation step is always skipped.

How

Change github.event_name == 'pull_request_target' to github.event_name == 'pull_request' in the if conditions of the validate-pr-title job to match the workflow's trigger event.

Testing

The workflow is now running.

Check failed when I set incorrect/unconventional PR title: https://github.com/aws/s2n-tls/actions/runs/22115920366/job/63923887211?pr=5744

Check succeed when I follow conventional PR title: https://github.com/aws/s2n-tls/actions/runs/22116045551/job/63924316532?pr=5744

Related

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions github-actions bot added the s2n-core team label Feb 17, 2026
@jouho jouho changed the title fix conventianal commit check ci: fix conventional commit check Feb 17, 2026
@jouho jouho marked this pull request as ready for review February 17, 2026 21:22
@jouho jouho requested review from dougch and jmayclin February 17, 2026 21:22
@jouho jouho enabled auto-merge February 17, 2026 21:46
@jouho jouho added this pull request to the merge queue Feb 17, 2026
Merged via the queue into aws:main with commit 9b175c9 Feb 17, 2026
54 checks passed
@jouho jouho deleted the cc branch February 17, 2026 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants