Skip to content

Make the Coveralls badge in the README work

Marek Szuba requested to merge bugfix/readme_badge_uris_master into master

Created by: mkszuba

Description

Update Coveralls URIs in README.md so to get the badge working.

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