Skip to content

ci: add package verification check#791

Open
vdusek wants to merge 7 commits into
masterfrom
ci/verify-built-package
Open

ci: add package verification check#791
vdusek wants to merge 7 commits into
masterfrom
ci/verify-built-package

Conversation

@vdusek
Copy link
Copy Markdown
Contributor

@vdusek vdusek commented May 13, 2026

Summary

Adds an end-to-end check that the built sdist and wheel install cleanly, ship the expected sources and data files, and that import apify_client works — via the shared apify/workflows/python-package-check action.

Wired into every PR via _check_package.yaml, and into both stable and beta release workflows so the exact artifact about to hit PyPI is verified first.

Verified locally

Built dist/ and ran the verification — all checks pass (apify_client==2.5.1, sdist + wheel both install cleanly into a fresh Python 3.14 venv).

Notes

Adds an end-to-end check that the built sdist and wheel install
cleanly and contain the expected sources via the shared
apify/workflows/python-package-check action. Runs on every PR and
inside the release flow between prepare-pypi-distribution and the
PyPI publish step, guarding against silent build regressions.
@vdusek vdusek added adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. labels May 13, 2026
@vdusek vdusek self-assigned this May 13, 2026
@github-actions github-actions Bot added this to the 140th sprint - Tooling team milestone May 13, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.71%. Comparing base (d1bd5a3) to head (bbd5ca5).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #791      +/-   ##
==========================================
- Coverage   97.73%   97.71%   -0.03%     
==========================================
  Files          48       48              
  Lines        4860     4860              
==========================================
- Hits         4750     4749       -1     
- Misses        110      111       +1     
Flag Coverage Δ
integration 93.97% <ø> (+0.30%) ⬆️
unit 86.19% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

vdusek and others added 6 commits May 14, 2026 11:04
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Replaces the local _check_package.yaml with a direct call to the
shared reusable workflow.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…ction

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vdusek vdusek requested a review from janbuchar May 14, 2026 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants