Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exclude compiler-generated code from coverage analysis #1546

Closed

Conversation

sharwell
Copy link
Member

No description provided.

@codecov-io
Copy link

Current coverage is 87.16%

Merging #1546 into master will increase coverage by +9.88% as of 2150137

@@            master   #1546   diff @@
======================================
  Files          530     460      -70
  Stmts        32148   15024   -17124
  Branches      8794    1890    -6904
  Methods          0       0         
======================================
- Hit          24844   13095   -11749
+ Partial       5135     706    -4429
+ Missed        2169    1223     -946

Review entire Coverage Diff as of 2150137

Powered by Codecov. Updated on successful CI builds.

@pdelvo
Copy link
Member

pdelvo commented Sep 26, 2015

👍

@sharwell
Copy link
Member Author

I'm not so sure this is what we want. The goal was to exclude the unreachable portions of the state machine generated for async methods, but it turned out to completely exclude all async methods as well as iterator blocks. I'm not sure what the right approach is yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants