add TRIO113, nursery.start_soon() in async context manager

[jakkdl]

Fixes #57

Quick and dirty:

• the message probably needs bettering.
• require that the object start_soon is called on is called "nursery"?
• require that it's also inside a with?
• could probs extend tests, but I think I covered most cases.
• maybe doesn't need it's own class, but since speed hasn't been an issue /shrug
• add config option

[jakkdl]

huh, test failures looks like it's an issue with the CI?

[jakkdl]

huh, test failures looks like it's an issue with the CI?

nvm, the warning was unrelated. But I'd forgotten to push the test file :upside_down_face:

[jakkdl]

Should be pretty much done now, other than what to do with the function signature on __aenter__

[Zac-HD]

👀 looks like flake8 6.0.0 just came out with a new required argument in some logic we use for tests... fixing now.

[jakkdl]

Looks good :rocket:

[Zac-HD]

Found two bugs with it right away 💪

Also looks like chasing down some similar patterns will result in follow-ups, so expect more issues next week 😅

[jakkdl]

Awesome!

Haha, wonderful :grin:

[Zac-HD]

Follow up PR please?

The other thing it could cover is the idiom of using [functools]partial() to pass keyword arguments through - it'd be great to check nursery.start_soon(partial(trio.run_process, cmd, shell=True)) by "seeing through" the call to partial.

[jakkdl]

Will do :+1: