romeovs / lcov-reporter-action

Comments a pull request with the code coverage generated by your tests.
MIT License
162 stars 100 forks source link

fix Empty value for parameter 'issue_number': undefined #61

Open klausbadelt opened 1 year ago

klausbadelt commented 1 year ago

The action fails on commits to a branch - as opposed to a PR - with RequestError [HttpError]: Empty value for parameter 'issue_number': undefined.

Fixes #33 .

This was also fixed by others, like https://github.com/CayabaCare/lcov-reporter-action/commit/c81818cd90cb309ec0c3d07feec613ccb1ce92c9 - but I couldn't find a current PR. Here it is.

ylhan commented 1 year ago

+1 this is breaking on merge in my CI

snggeng commented 10 months ago

hi wondering if this will get merged since it's been approved? It's breaking my CI as well. thanks!

snggeng commented 8 months ago

bumping this!

snggeng commented 6 months ago

@klausbadelt sorry to keep pinging you on this. wondering when this will get merged?

klausbadelt commented 6 months ago

@klausbadelt sorry to keep pinging you on this. wondering when this will get merged?

HI @snggeng - I'm not a a repo owner and can't merge. Only @romeovs and his contributors can.

stanleygtrillion commented 6 days ago

released a fork for this repo that can be used here: https://github.com/marketplace/actions/lcov-coverage-report uses: trillion-network/lcov-reporter-action@v0.4.0

also contains various upgrades and keeping up to the new API and library