Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow documentation builds on Windows #3227

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

A5rocks
Copy link
Contributor

@A5rocks A5rocks commented Mar 20, 2025

Fixes #3071

I'm not totally sure about all changes here. Some are likely only because my local installation of Python is 3.13.

Also this is definitely not the cleanest method of doing this :P

A5rocks added 2 commits March 21, 2025 05:40
@A5rocks A5rocks added the skip newsfragment Newsfragment is not required label Mar 20, 2025
Copy link

codecov bot commented Mar 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00000%. Comparing base (b6813ed) to head (6b92ded).

Additional details and impacted files
@@               Coverage Diff               @@
##                 main        #3227   +/-   ##
===============================================
  Coverage   100.00000%   100.00000%           
===============================================
  Files             124          124           
  Lines           18779        18788    +9     
  Branches         1269         1271    +2     
===============================================
+ Hits            18779        18788    +9     
Files with missing lines Coverage Δ
src/trio/_core/__init__.py 100.00000% <100.00000%> (ø)
src/trio/_core/_run.py 100.00000% <ø> (ø)
src/trio/_highlevel_socket.py 100.00000% <100.00000%> (ø)
src/trio/_path.py 100.00000% <100.00000%> (ø)
src/trio/_tests/test_unix_pipes.py 100.00000% <ø> (ø)
src/trio/_unix_pipes.py 100.00000% <ø> (ø)
src/trio/lowlevel.py 100.00000% <100.00000%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

A5rocks added 2 commits March 21, 2025 06:58
@TeamSpen210
Copy link
Contributor

Makes sense to me. I've always just ignored the FDStream error in logs when building, since I didn't need to test that bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip newsfragment Newsfragment is not required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow building docs on Windows
2 participants