search

5monkeys / cobertura-action

GitHub Action to report cobertura coverage
MIT License
63 stars 69 forks source link

fix: add missing report_name in `action.yml` #39

Closed swistakm closed 3 years ago

swistakm commented 3 years ago

Usage of this action with report_name input give following warning:

Warning: Unexpected input(s) 'report_name', valid inputs are ['repo_token', 'path', 'skip_covered', 'minimum_coverage', 'show_line', 'show_branch', 'show_class_names', 'show_missing', 'show_missing_max_length', 'only_changed_files', 'pull_request_number']

This request add missing input definition to action.yml manifest.

github-actions[bot] commented 3 years ago

File Coverage
All files 100% :white_check_mark:

Minimum allowed coverage is 75%

Generated by :monkey: cobertura-action against 8be90af1be9b5d3b06cd6f02509a450dcaa49cbb

kiriasCMNH commented 3 years ago

I am interested on this fix as well, as it doesnt allow me to have two separate comments per coverage report.

swistakm commented 3 years ago

@kiriasCMNH, TBH I've discovered this because I was trying to do exactly the same. I was trying to create two independent coverage report comments on for two tests suites (unit tests and API tests).

FYI It turned out that I could use the report_name input even though it wasn't included in the action.yml. I still had the warning but the thing worked.