Skip to content

migrate: Fix version ruleset copilot removal#535

Merged
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:no-copilot
Mar 11, 2026
Merged

migrate: Fix version ruleset copilot removal#535
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:no-copilot

Conversation

@llucax
Copy link
Contributor

@llucax llucax commented Mar 9, 2026

The v0.15.0 migration for the Protect version branches ruleset did not properly remove the Copilot review rule in existing repositories.

Add a dedicated migration step to detect and remove the copilot_code_review rule.

@github-actions github-actions bot added part:docs Affects the documentation part:cookiecutter Affects the generation of projects using cookiecutter labels Mar 9, 2026
@llucax
Copy link
Contributor Author

llucax commented Mar 9, 2026

Based on #534.

@llucax llucax added this to the v0.16.0 milestone Mar 9, 2026
@llucax llucax marked this pull request as ready for review March 10, 2026 11:22
@llucax llucax requested a review from a team as a code owner March 10, 2026 11:22
@llucax llucax requested review from florian-wagner-frequenz and removed request for a team March 10, 2026 11:22
@llucax llucax enabled auto-merge March 10, 2026 11:22
@llucax llucax self-assigned this Mar 10, 2026
Copy link
Contributor

@daniel-zullo-frequenz daniel-zullo-frequenz left a comment

Choose a reason for hiding this comment

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

A typo in the release notes, LGTM otherwise

@llucax llucax force-pushed the no-copilot branch 2 times, most recently from 5728c0c to ab6050c Compare March 11, 2026 11:14
@llucax
Copy link
Contributor Author

llucax commented Mar 11, 2026

Updated.

@llucax llucax added this pull request to the merge queue Mar 11, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Mar 11, 2026
The v0.15.0 migration for the Protect version branches ruleset did not
properly remove the Copilot review rule in existing repositories.

Add a dedicated migration step to detect and remove the
`copilot_code_review` rule.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax
Copy link
Contributor Author

llucax commented Mar 11, 2026

Rebased, need a new approval.

@llucax llucax added this pull request to the merge queue Mar 11, 2026
Merged via the queue into frequenz-floss:v0.x.x with commit 30bba68 Mar 11, 2026
6 checks passed
@llucax llucax deleted the no-copilot branch March 11, 2026 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:cookiecutter Affects the generation of projects using cookiecutter part:docs Affects the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants