FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Coverage & CI Setup Improvement by vmuriart · Pull Request #345 · pythonnet/pythonnet · GitHub

Coverage & CI Setup Improvement - #345

Merged
vmuriart merged 4 commits into
pythonnet:masterfrom
vmuriart:ci2
Jan 31, 2017
Merged

Coverage & CI Setup Improvement#345
vmuriart merged 4 commits into
pythonnet:masterfrom
vmuriart:ci2

Conversation

vmuriart commented Jan 31, 2017
edited
Loading

Copy link
Copy Markdown
Contributor

What does this implement/fix? Explain your changes.

  • Add coverage (cs code limited to windows, no mature coverage tools for mono)
  • Speed up AppVeyor by 50%, conda recipes built only when pull_request
  • Speeds up Travis ~30% by using containers
  • Simplifies and unfreezes mono setup

Does this close any currently open issues?

#237
#334

Any other comments?

Redid the previous pull request. (#335)

Sample coverage report.

codecov-io commented Jan 31, 2017
edited
Loading

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (master@ef133a3).

@@           Coverage Diff            @@
##             master    #345   +/-   ##
========================================
  Coverage          ?   61.1%           
========================================
  Files             ?      61           
  Lines             ?    5324           
  Branches          ?     897           
========================================
  Hits              ?    3253           
  Misses            ?    1832           
  Partials          ?     239

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 ef133a3...1145a44. Read the comment docs.

vmuriart merged commit 8035d67 into pythonnet:master Jan 31, 2017
vmuriart deleted the ci2 branch January 31, 2017 15:33
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL