Closed xurizaemon closed 1 year ago
Great, will do! Thanks.
Hi @sduenas - rebased to account for #801, removed CHANGELOG entry which has caused the check-changelog check to fail as it should 😁
Sidebar: Is that Coveralls check accurate? I'm not clear how this PR seemed to at first increase, and now decrease, coverage. But I'm not familiar with Coveralls so my expectation that it reflects test coverage vs LoC may be incorrect.
Sidebar: Is that Coveralls check accurate? I'm not clear how this PR seemed to at first increase, and now decrease, coverage. But I'm not familiar with Coveralls so my expectation that it reflects test coverage vs LoC may be incorrect.
Unless the change of coverage is big, I usually ignore it. Sometimes it doesn't calculate very well small changes or when there aren't changes at all.
Merged
Thanks for your contribution @xurizaemon
There is no documentation for the GitLab data source. This is based off the GitHub docs but I've reviewed it and believe it is correct for GitLab.
Additional minor changes here:
perceval gitlab --help
, other groups are lowercase whileGitLab
was not.This PR would need updating if #801 landed first, and I will update whichever is required when one or the other lands😁