Skip to content

th#1918: .yml -> .yaml in python-gen-worker (PyPI trusted-publisher filename must be updated) - #784

Merged
PaulFidika merged 1 commit into
masterfrom
th1918-yml-to-yaml
Aug 14, 2026
Merged

th#1918: .yml -> .yaml in python-gen-worker (PyPI trusted-publisher filename must be updated)#784
PaulFidika merged 1 commit into
masterfrom
th1918-yml-to-yaml

Conversation

@PaulFidika

Copy link
Copy Markdown
Contributor

Paul, 2026-08-13: "please rename all .yml -> .yaml; I'm going with that naming now instead."
Fleet-wide campaign, tracked as th#1918; one mechanical PR per repo.

⚠️ ACTION REQUIRED ON PyPI BEFORE THE NEXT gen-worker TAG

PyPI trusted publishing binds to the workflow FILENAME. The trusted-publisher entry for
the gen-worker project names publish.yml. Once this merges, a tag push mints an OIDC
token whose workflow_ref says publish.yaml, and PyPI will refuse the upload — the
publish job goes red at the last step, after CI, build and provenance have all passed.

Fix (Paul, one edit, ~30 seconds): PyPI → gen-workerManagePublishing → edit the
GitHub publisher's Workflow name from publish.yml to publish.yaml. Do it before the
next release cut; nothing else in the release procedure changes.

This is the only OIDC-bound workflow in the cozy repos other than cozy-eval, whose
publish.ymlpublish.yaml rename is already merged — so the cozy-eval trusted-publisher
entry needs the same edit and is broken right now until it gets one.

What changed

Renames: ci, native-kernels, proto-contract, publish workflows and Taskfile.
No name: or job id changes, so fast gates / tests and the merge queue are untouched.

Load-bearing references updated in the same commit:

  • native-kernels and proto-contract self-referencing paths: filters — a stale entry
    there would stop each workflow triggering on edits to itself.
  • scripts/assert_ci_proof.py: the Actions API path actions/workflows/ci.yaml/runs.
    Note GitHub treats a renamed workflow as a new one, so this query returns only runs from
    after the rename — the first post-merge ci run repopulates it, which happens on this
    merge itself.
  • scripts/lint_mypy_ratchet.py workflow path.
  • tests/test_publish_proof_gate_pgw1191.py path constant and the stderr assertion.
  • docs/releasing.md commands (gh workflow run ci.yaml --ref …), plus Taskfile and
    workflow comments.

CHANGELOG.md is deliberately untouched: its .yml mentions are dated historical records
of what those files were called at the time (and lanes do not edit the changelog).

@PaulFidika
PaulFidika enabled auto-merge August 14, 2026 01:54
Paul, 2026-08-13: "please rename all .yml -> .yaml; I'm going with that
naming now instead." Fleet umbrella th#1918; this is the pgw leg.

*** ACTION REQUIRED ON PyPI BEFORE THE NEXT TAG ***
PyPI trusted publishing binds to the workflow FILENAME. The trusted-publisher
entry for the `gen-worker` project names `publish.yml`; once this lands the
tag-push publish job mints an OIDC token for `publish.yaml` and PyPI will
REFUSE it. Edit the entry (PyPI -> gen-worker -> Manage -> Publishing) to
`publish.yaml` before cutting the next release.

Renames: ci/native-kernels/proto-contract/publish workflows and Taskfile.
No `name:` or job id changes, so `fast gates` / `tests` required contexts
and the merge queue are untouched.

Load-bearing references updated in the same commit:
- native-kernels + proto-contract self-referencing `paths:` filters
- scripts/assert_ci_proof.py Actions API path (actions/workflows/ci.yaml/runs)
- scripts/lint_mypy_ratchet.py workflow path
- tests/test_publish_proof_gate_pgw1191.py path + stderr assertion
- docs/releasing.md commands, Taskfile and workflow comments

CHANGELOG.md is left untouched: its `.yml` mentions are dated historical
records of what those files were called at the time.
@PaulFidika
PaulFidika added this pull request to the merge queue Aug 14, 2026
Merged via the queue into master with commit 3efd89c Aug 14, 2026
4 checks passed
@PaulFidika
PaulFidika deleted the th1918-yml-to-yaml branch August 14, 2026 02:48
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