Skip to content

promote(customer-priority-bump): apply 'priority' label instead of Priority field (D5) - #85

Merged
LukasWodka merged 1 commit into
mainfrom
promote/cpb-label-to-main
Jul 26, 2026
Merged

promote(customer-priority-bump): apply 'priority' label instead of Priority field (D5)#85
LukasWodka merged 1 commit into
mainfrom
promote/cpb-label-to-main

Conversation

@LukasWodka

@LukasWodka LukasWodka commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Promotes #84 to main so the live reusable workflow stops writing the soon-to-be-deleted Priority project field.

customer-priority-bump.yml runs from @main (that's what the per-repo callers reference), so the field-writing version was still the one executing in production. This copies develop's label-based version (gh issue edit --add-label priority) onto main verbatim.

Prerequisite for deleting the Priority project field (part of #1273 — field cleanup, D4/D5/D12: one priority source = the priority label).

Test plan

Part of #1273.


Note

Low Risk
Narrow CI automation change aligned with board cleanup; no app auth or data paths, though mis-timed merge could briefly conflict with deleting the Priority field.

Overview
Promotes the D5 behavior to main for the reusable customer-priority-bump workflow so production stops updating the deprecated Priority project field.

When from:customer is applied, the workflow now runs gh issue edit --add-label priority instead of GraphQL against Project V2 (field lookup, retry loop for project item registration, and updateProjectV2ItemFieldValue). Unused workflow env vars tied to the old path (ORG, PROJECT_NUMBER, PRIORITY_NAME) are removed; workflow_call inputs like target-priority remain but are no longer used by the step.

Reviewed by Cursor Bugbot for commit 9c1103e. Bugbot is set up for automated code reviews on this repo. Configure here.

@LukasWodka

Copy link
Copy Markdown
Contributor Author

bugbot run

@LukasWodka

Copy link
Copy Markdown
Contributor Author

👋 Heads-up — Code review queue is at 39 / 30

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 9c1103e. Configure here.

@LukasWodka

Copy link
Copy Markdown
Contributor Author

skip-fr-gate — written reason (RFC-0008 D27): legitimate release-mechanics promotion. This copies the already-reviewed #84 (customer-priority-bump label fix — merged to develop, Bugbot-clean) verbatim onto main, so the live @main reusable workflow stops writing the Priority project field ahead of its deletion (#1273 / D5). The single promotion commit carries no (#N) because it's a direct file copy, which is why the gate correctly flags it as unattributable. .github has no deployable environment to functional-review on, so the FR-ladder doesn't map — this is exactly the "legitimate release-mechanics commit" the gate names.

@LukasWodka LukasWodka added the skip-fr-gate Emergency override of the FR gate. Requires a written reason on the PR. label Jul 26, 2026
@LukasWodka LukasWodka closed this Jul 26, 2026
@LukasWodka LukasWodka reopened this Jul 26, 2026
@LukasWodka
LukasWodka merged commit 97ac1b7 into main Jul 26, 2026
11 of 12 checks passed
@LukasWodka
LukasWodka deleted the promote/cpb-label-to-main branch July 26, 2026 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-fr-gate Emergency override of the FR gate. Requires a written reason on the PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant