Fix Windows release pytest temp directory setup - #24
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThe release workflow for Windows now ensures the Flow diagram for Windows release pytest temp directory setupflowchart TD
A[Windows_release_job step] --> B["Run python -c import os; os.makedirs('.pytest-tmp', exist_ok=True)"]
B --> C["Run pytest -m not integration<br/>--basetemp .pytest-tmp/release<br/>-o cache_dir=.pytest-tmp/cache"]
C --> D[Subsequent build steps]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe release workflow now creates ChangesRelease test setup
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Root cause
The Windows release job passed a nested
--basetemp .pytest-tmp/releaseto pytest without creating its parent directory. Pytest 9.1.1 creates the final directory withparents=False, so every test usingtmp_pathfailed during fixture setup. The 186 errors were one cascading infrastructure failure, not independent test failures.Fix
Create
.pytest-tmpbefore running the release test command, matching the existing unit and coverage workflows.Validation
348 passed, 12 deselected.actionlintpassed.git diff --checkpassed.After merging, rerun the failed release workflow (or dispatch
releasemanually) to continue building and publishing version 0.2.1.Summary by Sourcery
CI: