Conversation
Pull Request Test Coverage Report for Build 1182
💛 - Coveralls |
|
@fantix Is it safe to merge? From the changelog, I assume some code to distinguish between |
|
It’s a bit weird how the tests passed when MagicStack/contextvars#11 is very much the same as aiocontextvars 0.2, but that PR really requires some more discussion before it becomes final. I’d like to see how the proposal works as aiocontextvars 0.2 in GINO and how people thinks about the approach in action. So after fixing the integration with 0.2, I’d like it merged before MagicStack/contextvars#11 have conclusion. And switching to contextvars shall be considered eventually after MagicStack/contextvars#2 is fixed. |
|
|
|
Ah yes, then it is safe to merge. I'll remove the |
This PR updates aiocontextvars from 0.1.2 to 0.2.0.
Changelog
0.2.0
Links