buildah icon indicating copy to clipboard operation
buildah copied to clipboard

Fix commit validation workflow

Open cevich opened this issue 3 months ago • 4 comments

What type of PR is this?

/kind other

What this PR does / why we need it:

Fix commit title-length checking workflow bug

How to verify it

The workflow will fail on this PR when the very long 'DO NOT MERGE:...' commit is included. But the workflow will pass when that commit is removed.

Which issue(s) this PR fixes:

None

Special notes for your reviewer:

Does this PR introduce a user-facing change?

None

cevich avatar Dec 10 '25 15:12 cevich

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: cevich Once this PR has been reviewed and has the lgtm label, please assign luap99 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci[bot] avatar Dec 10 '25 15:12 openshift-ci[bot]

Ref: Workflow log with the too-long "DO NOT MERGE: ..." commit 26a4c1c.

cevich avatar Dec 10 '25 15:12 cevich

Ephemeral COPR build failed. @containers/packit-build please check.

Ref: Workflow log without the too-long "DO NOT MERGE: ..."

cevich avatar Dec 10 '25 15:12 cevich

@cevich I like the idea, but the tests don't as of yet.

TomSweeneyRedHat avatar Dec 20 '25 01:12 TomSweeneyRedHat