coverallsapp / coverage-reporter

Self-contained, universal coverage uploader binary. Under development.
https://coveralls.io
MIT License
45 stars 14 forks source link

fix(semaphore): correct ENV variables #56

Closed jaredmoody closed 1 year ago

jaredmoody commented 1 year ago

and add additional options from available variables

Per semaphore docs, use the correct ENV variable names.

This partially addresses #55, in that the reporter should now read the correct environment variables on Semaphore, but does not address --service-job-id being ignored when --done is present.

The only thing I'm not sure about here is the service_build_url and service_job_url semantics mapping. On semaphore, the "job" is the individual process and the "workflow" is the entire build.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 4660313599


Totals Coverage Status
Change from base Build 4656075024: 0.05%
Covered Lines: 697
Relevant Lines: 771

💛 - Coveralls
jaredmoody commented 1 year ago

Great, made those updates.