Skip to content

ci: allow macOS prepublish gate 25 minutes - #285

Merged
ictechgy merged 2 commits into
mainfrom
ci/macos-prepublish-timeout-25
Aug 6, 2026
Merged

ci: allow macOS prepublish gate 25 minutes#285
ictechgy merged 2 commits into
mainfrom
ci/macos-prepublish-timeout-25

Conversation

@ictechgy

@ictechgy ictechgy commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • raise only the macOS prepublish step timeout from 18 to 25 minutes
  • pin the job-scoped workflow contract to the new budget
  • refresh the closed production-surface inventory digest for the workflow byte change
  • leave Ubuntu, release smoke, Swift, and npm publish budgets unchanged

Evidence

PR #283 run 31088395364 reached the existing hard timeout after 18 minutes while Linux 3.11 and 3.12 passed. Earlier successful macOS runs needed up to 16m26s, so the old budget had insufficient runner-variance headroom.

Verification

  • python3 -B -m unittest tests.test_workflows (8 passed)
  • python3 -B -m unittest tests.test_contextguard_stage2_feasibility (3 passed)
  • git diff --check

@ictechgy

ictechgy commented Aug 6, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The macOS prepublish release-gate timeout increases from 18 to 25 minutes. The workflow test expectation changes to match the new timeout.

Changes

macOS prepublish timeout

Layer / File(s) Summary
Update timeout and validation
.github/workflows/ci.yml, tests/test_workflows.py
The macOS prepublish step timeout and its test expectation change from 18 to 25 minutes.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the macOS prepublish gate timeout increase to 25 minutes.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/macos-prepublish-timeout-25

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

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Already reviewed.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@ictechgy
ictechgy merged commit d7d53b9 into main Aug 6, 2026
4 checks passed
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.

1 participant