A5rocks
> That should work, if it's in `sys.path` so pytest can import. Nevermind! I didn't realize pytest's sys.path schenanigans are the way they are. I'm surprised the test passes in...
Ohhhhh nooooo, it's installing `_trio_check_attrs_aliases.py` as a module into `site-packages` as part of the single wheel we make. That's quite the footgun!!!! As a knee-jerk reaction I think we should...
> The plugin script can't be anywhere in the `trio` package, because then trying to import it implicitly imports `trio` first, before we hooked anything. That's why I gave it...
Spotted: - It seems to be using the wrong path for Alpine (it includes the full path including the `.venv`) - It seems to be ignoring our specified coverage ignore...
First part is fixed
OK nevermind that fixes both issues, @webknjaz in case you want to copy these changes to your PR.
Turns out I can't modify that PR to add in a few commits. I guess codecov v5 isn't that urgent anyways though.
codecov failure doesn't make any sense and seems to be a recent thing. I'm inclined to believe it's some change in how they normalize file names but I don't think...
I can do the change to abort fn as a follow up -- it needs this change because the reason it's a function is because KI can be raised (which...
Looking at #3030 makes me unsure whether this will work with that (specifically pressing ctrl+c when some async code is already running seems like it would break with this). I'd...