Skip to content

Handle private repos in workflow migration#548

Merged
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:fix-perm-migrate
Mar 30, 2026
Merged

Handle private repos in workflow migration#548
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:fix-perm-migrate

Conversation

@llucax
Copy link
Copy Markdown
Contributor

@llucax llucax commented Mar 26, 2026

Treat the docs and PyPI publish patterns in ci.yaml as optional when the repository appears to be private, so migrations do not fail just because those jobs were removed intentionally.

When those publish jobs are still present in a private repository, warn with a manual step so maintainers can remove them after applying the workflow updates.

@llucax llucax requested a review from a team as a code owner March 26, 2026 13:13
@llucax llucax requested review from Marenz and removed request for a team March 26, 2026 13:13
@llucax llucax enabled auto-merge March 26, 2026 13:13
@github-actions github-actions Bot added part:tests Affects the unit, integration and performance (benchmarks) tests part:cookiecutter Affects the generation of projects using cookiecutter labels Mar 26, 2026
@llucax llucax added the cmd:skip-release-notes It is not necessary to update release notes for this PR label Mar 26, 2026
@llucax
Copy link
Copy Markdown
Contributor Author

llucax commented Mar 26, 2026

Skip release notes because this fixes/improves an unreleased migration script.

@llucax llucax force-pushed the fix-perm-migrate branch from ace7c0f to 5736687 Compare March 26, 2026 13:14
@llucax llucax added this to the v0.17.0 milestone Mar 26, 2026
Treat the docs and PyPI publish patterns in ci.yaml as optional when the
repository appears to be private, so migrations do not fail just because
those jobs were removed intentionally.

When those publish jobs are still present in a private repository, warn
with a manual step so maintainers can remove them after applying
the workflow updates.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax llucax force-pushed the fix-perm-migrate branch from 5736687 to f50a702 Compare March 26, 2026 13:41
@llucax llucax added the type:bug Something isn't working label Mar 27, 2026
@llucax llucax added this pull request to the merge queue Mar 30, 2026
Merged via the queue into frequenz-floss:v0.x.x with commit eec17e3 Mar 30, 2026
7 checks passed
@llucax llucax deleted the fix-perm-migrate branch March 30, 2026 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cmd:skip-release-notes It is not necessary to update release notes for this PR part:cookiecutter Affects the generation of projects using cookiecutter part:tests Affects the unit, integration and performance (benchmarks) tests type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants