Skip to content

Bump dependencies from commit d66c4f#3446

Merged
github-actions[bot] merged 5 commits into
mainfrom
autodeps/bump_from_d66c4f
Jun 1, 2026
Merged

Bump dependencies from commit d66c4f#3446
github-actions[bot] merged 5 commits into
mainfrom
autodeps/bump_from_d66c4f

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Jun 1, 2026

No description provided.

@github-actions github-actions Bot added the dependencies Pull requests that update a dependency file label Jun 1, 2026
@github-actions github-actions Bot enabled auto-merge (squash) June 1, 2026 02:35
@CoolCat467
Copy link
Copy Markdown
Member

Copying from #3444

Issue from "Context manager does not catch exceptions", should we add exception handling for those two places or ignore the error?

@A5rocks
Copy link
Copy Markdown
Contributor

A5rocks commented Jun 1, 2026

They already do handle errors, there's a context manager around them. IMO the error code is too noisy and we should ignore it. It raises an error if there's any cleanup code after the yield, like we have there. (or maybe just noqa, idk)

A5rocks and others added 2 commits June 1, 2026 13:50
noqa RUF075 since we already handle exceptions
@A5rocks A5rocks added the skip newsfragment Newsfragment is not required label Jun 1, 2026
@github-actions github-actions Bot merged commit bc70f9d into main Jun 1, 2026
42 checks passed
@github-actions github-actions Bot deleted the autodeps/bump_from_d66c4f branch June 1, 2026 05:04
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00000%. Comparing base (d66c4fb) to head (bdca009).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@               Coverage Diff               @@
##                 main        #3446   +/-   ##
===============================================
  Coverage   100.00000%   100.00000%           
===============================================
  Files             128          128           
  Lines           19419        19417    -2     
  Branches         1317         1317           
===============================================
- Hits            19419        19417    -2     
Files with missing lines Coverage Δ
src/trio/_core/_run.py 100.00000% <ø> (ø)
src/trio/_core/_tests/test_ki.py 100.00000% <ø> (ø)
src/trio/_tests/test_dtls.py 100.00000% <100.00000%> (ø)
src/trio/_tests/test_subprocess.py 100.00000% <100.00000%> (ø)
src/trio/_tests/tools/test_sync_requirements.py 100.00000% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file skip newsfragment Newsfragment is not required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants