Skip to content

ci: drop redundant Test PyPI upload step from release workflow - #355

Merged
blink1073 merged 1 commit into
Calysto:mainfrom
blink1073:no-testpypi
Aug 1, 2026
Merged

ci: drop redundant Test PyPI upload step from release workflow#355
blink1073 merged 1 commit into
Calysto:mainfrom
blink1073:no-testpypi

Conversation

@blink1073

Copy link
Copy Markdown
Contributor

References

Description

Implements item 1 of the CI/release workflow hardening checklist (based on calysto/metakernel#466): drops the "Upload package to Test PyPI" step from release.yml.

Items 2 and 3 of that checklist are already covered in this repo:

  • Static package checks: validate-pyproject already runs via pre-commit in the static job, and twine check --strict already runs as part of hynek/build-and-inspect-python-package (used by the build job on every push/PR).
  • Codecov fork-PR fix: already merged in b2c4686 ("ci: allow codecov uploads on fork PRs targeting Calysto/octave_kernel").

Changes

  • Remove the "Upload package to Test PyPI" step from release.yml. It doesn't catch anything the existing static build/package checks don't already catch, and it's one more external service in the critical path of every release.

Backwards-incompatible changes

None

Testing

  • just pre-commit
  • just typing

AI usage

  • Some or all of the content of this PR was generated by AI.
  • The human author has carefully reviewed this PR and run this code.
  • AI tools and models used: Claude Code

TestPyPI doesn't catch anything the static build/package checks
(validate-pyproject, twine check via build-and-inspect-python-package)
don't already catch, and it's one more external service in the
critical path of every release.
@blink1073
blink1073 enabled auto-merge (squash) August 1, 2026 12:43
@codecov

codecov Bot commented Aug 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.93%. Comparing base (cd925fa) to head (ff77784).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #355   +/-   ##
=======================================
  Coverage   92.93%   92.93%           
=======================================
  Files           6        6           
  Lines         439      439           
=======================================
  Hits          408      408           
  Misses         31       31           
Flag Coverage Δ
linux 92.71% <ø> (ø)
windows 92.93% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@blink1073
blink1073 merged commit fa68d09 into Calysto:main Aug 1, 2026
30 of 31 checks passed
@blink1073
blink1073 deleted the no-testpypi branch August 1, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant