Skip to content

CI: Add PR title verifier workflow#277

Open
rubambiza wants to merge 6 commits intokagenti:mainfrom
rubambiza:ci/add-pr-verifier
Open

CI: Add PR title verifier workflow#277
rubambiza wants to merge 6 commits intokagenti:mainfrom
rubambiza:ci/add-pr-verifier

Conversation

@rubambiza
Copy link
Copy Markdown
Contributor

@rubambiza rubambiza commented Apr 28, 2026

Summary

  • Adds a caller workflow that invokes the org-level reusable PR title verifier from kagenti/.github
  • Enforces PR title prefix conventions (Feat, Fix, Docs, etc.) on all pull requests
  • No changes to existing workflows or repo configuration

@rubambiza
ci: Add PR verifier caller workflow
a7f9e0b 
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
@rubambiza
ci: Pin reusable workflow to SHA
8831a40 
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
@rubambiza
ci: Update pinned SHA to include action fix
8276dce 
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
@rubambiza
Copy link
Copy Markdown
Contributor Author

rubambiza commented Apr 28, 2026

Update: The reusable workflow in kagenti/.github is now live (PR #66 merged), so this caller workflow will fire correctly as soon as this PR is merged.

However, the pinned SHA in this workflow will need a final bump once kagenti/.github#67 merges. That PR adds FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true to suppress a Node.js 20 deprecation warning from the action-pr-title action (GitHub is removing Node 20 runner support on September 16, 2026). The action itself is safe for Node 24 (only uses @actions/core and @actions/github).

Once #67 merges, I'll push an updated SHA pin to this branch.

@xjacka xjacka mentioned this pull request Apr 29, 2026
Open
@rubambiza
ci: Fix trailing newline in pr-verifier.yml
d527b30 
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
esnible
esnible reviewed Apr 29, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@esnible esnible left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would love to see comments on this YAML. Not a lot -- but perhaps the description of this PR belongs also on the YAML?

@rubambiza
ci: Bump pinned SHA and rename job to verify-pr-title
29a0f70 
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
@rubambiza
ci: Add top-level permissions restriction
2798537 
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
@clawgenti clawgenti mentioned this pull request May 4, 2026
Closed
@xjacka xjacka mentioned this pull request May 4, 2026
Open
@clawgenti clawgenti mentioned this pull request May 4, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@esnible esnible esnible left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

Kagenti Issue Prioritization
Status: Backlog

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rubambiza @esnible