Skip to content

🌱 Make the synced operator PR pass title and DCO checks - #219

Open
djzager wants to merge 1 commit into
konveyor:mainfrom
djzager:sync-seedling-prefix
Open

🌱 Make the synced operator PR pass title and DCO checks#219
djzager wants to merge 1 commit into
konveyor:mainfrom
djzager:sync-seedling-prefix

Conversation

@djzager

@djzager djzager commented Sep 4, 2026

Copy link
Copy Markdown
Member

The sync-operator.yml workflow produced a konveyor/operator PR that failed two of that repo's required checks:

  1. PR-title check — the commit/title used a :robot: prefix, which is not in konveyor's allowed emoji-alias set (:warning:, :sparkles:, :bug:, :book:, :seedling:, :ghost:). An automated manifest sync is infra → :seedling:.
  2. DCO — the bot commit was unsigned. Add git commit -s so the Signed-off-by matches the configured konveyor bot identity.

Follow-up to #215. Merging this re-triggers the sync (it edits sync-operator.yml, a trigger path), which force-pushes the existing sync PR (konveyor/operator#605) with a corrected, signed :seedling: commit — clearing both checks there automatically. No changelog fragment (:seedling:).

@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: f3de5caf-0378-4347-93b3-4a02573e8193


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@djzager
djzager force-pushed the sync-seedling-prefix branch from 41369d9 to bde3a8d Compare September 4, 2026 17:11
@djzager djzager changed the title 🌱 Use 🌱 prefix for synced operator manifests 🌱 Make the synced operator PR pass title and DCO checks Sep 4, 2026
The sync-operator workflow's commit and PR title used a 🤖 prefix, which
is not in konveyor's allowed emoji-alias set (⚠️, ✨, 🐛,
📖, 🌱, 👻) and fails the operator repo's PR-title check; an
automated manifest sync is infra, so use 🌱. The commit was also
unsigned, so the operator's DCO check rejected it -- add git commit -s so the
Signed-off-by matches the configured konveyor bot identity.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: David Zager <david.j.zager@gmail.com>
@djzager
djzager force-pushed the sync-seedling-prefix branch from bde3a8d to 66f66d5 Compare September 4, 2026 20: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