Skip to content

Two small followups to gh-228#232

Merged
remleduff merged 2 commits intopython-trio:masterfrom
njsmith:asynccontextmanager-fixes
Jun 21, 2017
Merged

Two small followups to gh-228#232
remleduff merged 2 commits intopython-trio:masterfrom
njsmith:asynccontextmanager-fixes

Conversation

@njsmith
Copy link
Member

@njsmith njsmith commented Jun 21, 2017

No description provided.

@codecov
Copy link

codecov bot commented Jun 21, 2017

Codecov Report

Merging #232 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #232      +/-   ##
==========================================
+ Coverage   99.09%   99.09%   +<.01%     
==========================================
  Files          63       63              
  Lines        8466     8472       +6     
  Branches      610      610              
==========================================
+ Hits         8389     8395       +6     
  Misses         61       61              
  Partials       16       16
Impacted Files Coverage Δ
trio/tests/test_util.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 979d603...670dfa9. Read the comment docs.

@kyrias
Copy link
Contributor

kyrias commented Jun 21, 2017

LGTM. Though looking at this diff in GitHub reminds me, I wonder if they use git to create the diff, or something of their own. Would be nice if it could detect async functions as a context header marker thing, rather than just using the last non-async function...

@njsmith
Copy link
Member Author

njsmith commented Jun 21, 2017

@kyrias: oh, huh, that's a good point. Probably worth emailing support@github about – do you want to do that? They've been pretty responsive in my experience.

@remleduff remleduff merged commit 4b9466c into python-trio:master Jun 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants