search

python-trio / trio

Trio – a friendly Python library for async concurrency and I/O
https://trio.readthedocs.io
Other
6.22k stars 342 forks source link

gh-3108: avoid materializing f_locals by using weakrefs to code objec… #3109

Closed graingert closed 1 month ago

graingert commented 1 month ago

…ts instead

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.51%. Comparing base (2a66a0d) to head (e04b3d8).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #3109 +/- ## ========================================== - Coverage 99.58% 99.51% -0.08% ========================================== Files 121 121 Lines 18166 18172 +6 Branches 3275 3272 -3 ========================================== - Hits 18091 18084 -7 - Misses 52 65 +13 Partials 23 23 ``` | [Files with missing lines](https://app.codecov.io/gh/python-trio/trio/pull/3109?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=python-trio) | Coverage Δ | | |---|---|---| | [src/trio/\_core/\_ki.py](https://app.codecov.io/gh/python-trio/trio/pull/3109?src=pr&el=tree&filepath=src%2Ftrio%2F_core%2F_ki.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=python-trio#diff-c3JjL3RyaW8vX2NvcmUvX2tpLnB5) | `100.00% <100.00%> (ø)` | | | [src/trio/\_core/\_run.py](https://app.codecov.io/gh/python-trio/trio/pull/3109?src=pr&el=tree&filepath=src%2Ftrio%2F_core%2F_run.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=python-trio#diff-c3JjL3RyaW8vX2NvcmUvX3J1bi5weQ==) | `99.02% <100.00%> (+<0.01%)` | :arrow_up: | | [src/trio/\_core/\_tests/test\_run.py](https://app.codecov.io/gh/python-trio/trio/pull/3109?src=pr&el=tree&filepath=src%2Ftrio%2F_core%2F_tests%2Ftest_run.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=python-trio#diff-c3JjL3RyaW8vX2NvcmUvX3Rlc3RzL3Rlc3RfcnVuLnB5) | `100.00% <100.00%> (ø)` | | | [src/trio/\_tools/gen\_exports.py](https://app.codecov.io/gh/python-trio/trio/pull/3109?src=pr&el=tree&filepath=src%2Ftrio%2F_tools%2Fgen_exports.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=python-trio#diff-c3JjL3RyaW8vX3Rvb2xzL2dlbl9leHBvcnRzLnB5) | `95.93% <100.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/python-trio/trio/pull/3109/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=python-trio)
graingert commented 1 month ago

closing this as I pushed directly to trio rather than my own fork and I'm now getting multiple CI runs