search

fayland / perl-Dist-Zilla-Plugin-GitHubREADME-Badge

Dist::Zilla - add badges to github README.md
Other
6 stars 10 forks source link

coveralls.io now uses a different URL structure #13

Closed mcmillhj closed 8 years ago

mcmillhj commented 8 years ago

coveralls.io appears to have changed the url structure that maps repos to coverage reports. They added another level between the domain and username, and replaced the /r/ with /github/.

e.g:

[![Coverage Status](https://coveralls.io/repos/mcmillhj/WUPHF/badge.svg?branch=master)](https://coveralls.io/r/mcmillhj/WUPHF?branch=master)

versus

[![Coverage Status](https://coveralls.io/repos/github/mcmillhj/WUPHF/badge.svg?branch=master)](https://coveralls.io/github/mcmillhj/WUPHF?branch=master)

I noticed this when I made a new Dist::Zilla repo this afternoon and figured I would send in a PR since it was a small change.

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 539fe348cf515abcb7365e38c9e00a6aaa8082ad on mcmillhj:enhancement/change-coveralls-url into d45e021f0faaf86e3e20761b277e2b341b6566d0 on fayland:master.

mcmillhj commented 8 years ago

Hmm, my mistake. It appears that there was just a delay in processing. Both of the URLs appear to point to the correct coverage reports.