Skip to content

ci: pass bumped version and post-bump ref to docs jobs in release flow#806

Merged
vdusek merged 2 commits into
masterfrom
ci/release-version-fix
May 20, 2026
Merged

ci: pass bumped version and post-bump ref to docs jobs in release flow#806
vdusek merged 2 commits into
masterfrom
ci/release-version-fix

Conversation

@vdusek
Copy link
Copy Markdown
Contributor

@vdusek vdusek commented May 20, 2026

Summary

The v3.0.0 release versioned the docs as 2.5 instead of 3.0 (overwriting v2.5 content with v3). Root cause: manual_version_docs.yaml reads uv version --short from a checkout of the workflow's dispatch ref (pre-bump), so the job sees the old version. Run #26156928273 logged Version: 2.5.1, Major.Minor: 2.5, Major: 2; bad output is commit e0a0bc8.

Fix

  • manual_version_docs.yaml — optional version_number input; snapshot step uses it when set, falls back to uv version --short for manual dispatch.

Companion data-restore PR: #808.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added this to the 141st sprint - Tooling team milestone May 20, 2026
@github-actions github-actions Bot added the t-tooling Issues with this label are in the ownership of the tooling team. label May 20, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.52%. Comparing base (80a5974) to head (d15cec6).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #806      +/-   ##
==========================================
+ Coverage   94.50%   94.52%   +0.02%     
==========================================
  Files          48       48              
  Lines        4916     4916              
==========================================
+ Hits         4646     4647       +1     
+ Misses        270      269       -1     
Flag Coverage Δ
integration 90.82% <ø> (ø)
unit 83.03% <ø> (ø)

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 vdusek added the adhoc Ad-hoc unplanned task added during the sprint. label May 20, 2026
@vdusek vdusek requested a review from janbuchar May 20, 2026 11:56
@vdusek vdusek marked this pull request as draft May 20, 2026 12:26
@vdusek vdusek removed the request for review from janbuchar May 20, 2026 12:26
@vdusek vdusek force-pushed the ci/release-version-fix branch from 5dd6251 to d15cec6 Compare May 20, 2026 12:31
@vdusek vdusek requested a review from janbuchar May 20, 2026 12:32
@vdusek vdusek marked this pull request as ready for review May 20, 2026 12:32
@vdusek vdusek merged commit 757a1fe into master May 20, 2026
27 checks passed
@vdusek vdusek deleted the ci/release-version-fix branch May 20, 2026 12:51
vdusek added a commit to apify/apify-sdk-python that referenced this pull request May 20, 2026
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.

3 participants