search

codacy / codacy-coverage-reporter-action

GitHub Action for the codacy-coverage-reporter
Other
56 stars 15 forks source link

Add option to force coverage parser #54

Closed rageagainsthepc closed 2 years ago

rageagainsthepc commented 2 years ago

This PR adds the possibility to force the coverage parser in order to allow coverage uploads for languages like golang.

rageagainsthepc commented 2 years ago

@mrfyda @lolgab Hi! Is there a reason this hasn't been reviewed yet? Or did you guys simply not see it?

mrfyda commented 2 years ago

@prcr can you have a look?

machadoit commented 2 years ago

Hi there! There is something minor to match the the other variables, which is the missing default. And I think we need to have a branch on the repo to run the pipeline successfully, so I may need to create a new PR.

Also, thank you very much for the contribution!

machadoit commented 2 years ago

@rageagainsthepc created https://github.com/codacy/codacy-coverage-reporter-action/pull/56 as follow up to be able to run the pipeline and fix the case when the parser is not passed. Once the that PR is merged, this one should be detected as merged, as I used your commit on the PR

rageagainsthepc commented 2 years ago

Whoopsie, I did not see the -u flag at the beginning of the script, my bad.

machadoit commented 2 years ago

Done and available at starting at v1.2.0