search

codacy / codacy-coverage-reporter-action

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

[CY-4907] Question - Mark as final command #52

Closed c-renaud-dpi closed 3 years ago

c-renaud-dpi commented 3 years ago

Hi there,

Any thoughts on adding the final command when submitting for multiple code coverage reports?

https://docs.codacy.com/coverage-reporter/#uploading-coverage image

github-actions[bot] commented 3 years ago

Internal ticket created : CY-4907

franciscodua commented 3 years ago

Hi @c-renaud

Ok so, here's my 2-cents on this.

The action is supposed to cater to simple use cases, and we didn't want to have it too bloated with functionality (betting on simplicity). And in the case of multiple reports, the idea is to include all at a time (which should be the main use case for the action).

For more advanced use cases, such as uploading multiple reports at different times, we suggest using the script directly.

But, this is my current view on the subject. Depending on the feedback we get, we can revisit these decisions we made.

Does this make sense? Or do you think I'm missing something?

franciscodua commented 3 years ago

Closing this issue due to inactivity. Feel free to re-open it if there's new information to add here.

c-renaud-dpi commented 3 years ago

Thanks @franciscodua , sorry for the lack of response. Agree on your thought process and will use the script directly for our scenario. Thanks again!