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

Update coverage badge #32

Merged
merged 1 commit into from
May 18, 2017
Merged

Update coverage badge #32

merged 1 commit into from
May 18, 2017

Conversation

lbenet
Copy link
Member

@lbenet lbenet commented May 18, 2017

Doing it this way to trigger travis and see if the problem with the docs persist.

Doing it this way to trigger travis
@coveralls
Copy link

coveralls commented May 18, 2017

Coverage Status

Changes Unknown when pulling 21be50b on coverage_badge into ** on master**.

@codecov-io
Copy link

codecov-io commented May 18, 2017

Codecov Report

Merging #32 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   92.14%   92.14%           
=======================================
  Files          22       22           
  Lines         917      917           
=======================================
  Hits          845      845           
  Misses         72       72

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 26470ff...21be50b. Read the comment docs.

@lbenet
Copy link
Member Author

lbenet commented May 18, 2017

Here I have only fixed the coveralls badge. The problem was actually not related to the badge itself, but to the settings of coveralls, which I have now fixed.

I have also deployed the ssh keys and set them in travis (this is related to #31, this comment). Documenter should build the documentation now, as far as I can see, once this is merged. Note that the docs should be build after merging (or committing to master), and only from the travis-build using linux and 0.5.

@dpsanders Shall we merge this?

@dpsanders dpsanders merged commit 96cee03 into master May 18, 2017
@dpsanders
Copy link
Member

Thanks, let's hope this works!

@lbenet
Copy link
Member Author

lbenet commented May 18, 2017

It seems everything (badge and docs) finally work: see here.

The branch gh-pages needs to be cleaned, and perhaps we should tag a new patch version (0.9.1) , so the stable docs are also generated. (Also, the gh-pages branch will need to be cleaned to remove the old docs.)

@lbenet
Copy link
Member Author

lbenet commented May 18, 2017

@dpsanders Do you agree about tagging a patch (0.9.1) version for IntervalArithmetic? The idea is simply to generate the stable docs, and then latter clean-up the gh-pages branch.

@dpsanders
Copy link
Member

Sure, go ahead.

@lbenet lbenet deleted the coverage_badge branch May 18, 2017 20:31
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.

4 participants