fix(ci): fix PyPI publish action input and enable skip-existing - #31
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 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. Comment |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThe release workflow now uses the supported PyPI token username input and skips artifacts that already exist, preventing workflow warnings and re-publish failures during release retries. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've reviewed your changes and they look great!
Sourcery assessment
Needs a human reviewer. If the corrected publish input or skip-existing behavior is wrong, the workflow could publish or accept an incorrect package state in PyPI that remains after reverting the workflow. The impact is bounded and can generally be mitigated by yanking the release and rerunning the publication, but consumers may already have downloaded the package.
What changed / why
.github/workflows/release.yml, updatepypa/gh-action-pypi-publishinput fromusernametouser: __token__and setskip-existing: true.usernameis an invalid input ingh-action-pypi-publish@release/v1that generated workflow warnings, andskip-existing: trueprevents 400 Bad Request errors when re-publishing or re-running a release whose artifacts are already on PyPI.AI assistance disclosure
ai-assistedlabel should be applied.Summary by Sourcery
Fix the release workflow so PyPI publishing succeeds without invalid-input warnings or failures when artifacts have already been uploaded.
Bug Fixes:
CI: