Skip to content

Make the Coveralls badge in the README work [release/97]

Marek Szuba requested to merge bugfix/readme_bugfix_uris_rel97 into release/97

Created by: mkszuba

Description

Update Coveralls URIs in README.md so to get the badge working. This is a backport of #385 to release/97 so that we aren't stuck with "coverage: unknown" on the main GitHub page of this repository all the way until the release of version 98.

Use case

Our Coveralls badge as it is keeps saying "unknown" because it doesn't specify the branch in the URI, which apparently is necessary in case of Coveralls (and recommended for Travis as well). Moreover, the URI itself might be outdated.

Benefits

Actually get the badge to work.

Possible Drawbacks

Keeping it working for release branches will require updating the README as part of the release procedure.

Testing

Have you added/modified unit tests to test the changes?

No.

If so, do the tests pass/fail?

N/A

Have you run the entire test suite and no regression was detected?

No, this is a change to documentation - and a quantitative one at that.

Merge request reports