Skip to content

CI: Add PR title verifier workflow#315

Merged
rubambiza merged 6 commits into
kagenti:mainfrom
rubambiza:ci/add-pr-verifier
May 7, 2026
Merged

CI: Add PR title verifier workflow#315
rubambiza merged 6 commits into
kagenti:mainfrom
rubambiza:ci/add-pr-verifier

Conversation

@rubambiza

Copy link
Copy Markdown
Contributor

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

Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
@rubambiza

Copy link
Copy Markdown
Contributor Author

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.

Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>

@ChristianZaccaria ChristianZaccaria left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm !!

@cwiklik cwiklik left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Clean caller workflow — pinned to SHA, permissions: {}, proper trigger types. No issues.

Areas reviewed: CI/GitHub Actions
Commits: 6, all signed-off
CI: all passing

@rubambiza rubambiza merged commit f858a70 into kagenti:main May 7, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Kagenti Issue Prioritization May 7, 2026
@rubambiza rubambiza deleted the ci/add-pr-verifier branch May 7, 2026 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants