Skip to content

ci: remove Test PyPI release workflow - #76

Merged
bhimrazy merged 1 commit into
mainfrom
ci/remove-test-pypi-release
Oct 6, 2026
Merged

bhimrazy merged 1 commit into
mainfrom
ci/remove-test-pypi-release

Conversation

@bhimrazy

@bhimrazy bhimrazy commented Oct 6, 2026 •

Copy link
Copy Markdown
Owner

What does this PR do?

Removes .github/workflows/release-test-pypi.yml. release-pypi.yml already builds the package, runs twine check --strict and the wheel/sdist smoke tests on every push to main, and publishes on release, so the only thing dropped is the upload to test.pypi.org.

  • Nothing else in the repo references this workflow
Follow-up cleanup (manual, optional)
  • Delete the testpypi environment under repo Settings → Environments
  • Remove the Trusted Publisher entry for this repo on test.pypi.org

Summary by CodeRabbit

  • Chores
    • Removed the automated Test PyPI release workflow. Test distributions are no longer built, checked, and published through this workflow when a release is created or a manual run is requested.

release-pypi.yml already builds, twine-checks and smoke-tests the package on
every push to main, so the Test PyPI upload is the only thing lost.
@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 159b63c3-9f20-40df-b465-6f02a2e5d807
📥 Commits

Reviewing files that changed from the base of the PR and between 45b5c3f and 055f5c6.

📒 Files selected for processing (1)
  • .github/workflows/release-test-pypi.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/release-test-pypi.yml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 1 remain after this review.


📝 Walkthrough

Walkthrough

The Test PyPI release workflow was deleted. It previously built and checked distribution artifacts, smoke-tested them, and published them to Test PyPI for releases and manual dispatches.

Changes

Test PyPI Workflow Removal

Layer / File(s) Summary
Remove Test PyPI release workflow
.github/workflows/release-test-pypi.yml
The workflow and its build, artifact validation, smoke-test, attestation, and Test PyPI publishing jobs were removed.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 055f5

The production release workflow retains its package checks and publishing path. No actionable merge-blocking risk was established.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely identifies the removal of the Test PyPI release workflow.
Description check ✅ Passed The description explains what the PR removes and why, and includes optional follow-up cleanup notes. It does not include a related issue link, but the main description is complete.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov-commenter

codecov-commenter commented Oct 6, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.33%. Comparing base (45b5c3f) to head (055f5c6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage   93.33%   93.33%           
=======================================
  Files          12       12           
  Lines         210      210           
=======================================
  Hits          196      196           
  Misses         14       14           
Flag Coverage Δ
receipt_ocr 89.34% <ø> (ø)
tesseract_ocr 98.86% <ø> (ø)

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

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@bhimrazy
bhimrazy merged commit fdcefb6 into main Oct 6, 2026
25 checks passed
@bhimrazy
bhimrazy deleted the ci/remove-test-pypi-release branch October 6, 2026 09:02
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.

2 participants