nickmerwin / node-coveralls

lcov posting to coveralls.io
Other
1.07k stars 256 forks source link

Azure pipelines branch name with slashes #244

Open KagamiChan opened 4 years ago

KagamiChan commented 4 years ago

Hi I'm trying using coveralls with Azure pipelines. The problem I met is that if I have a slash in branch name, then the coveralls report will point to a wrong branch because in Azure pipelines, Build.SourceBranchName will only take the last part.

I'm not sure if coveralls could accept a "full path" like refs/heads/foo/bar, if so we could use Build.SourceBranch instead

KagamiChan commented 4 years ago

tested with a fork and it runs https://github.com/KagamiChan/node-coveralls

I could file a PR if you think it is good

jtwebman commented 2 years ago

Since this library doesn't seem to be supported anymore I fix a bunch of things on a fork if you want to check it out and are still pulling the library into your packages: https://github.com/jtwebman/coveralls-next I would be willing to pull the fix into my fork.