mikepenz / action-junit-report

Reports junit test results as GitHub Pull Request Check
https://blog.mikepenz.dev
Apache License 2.0
314 stars 117 forks source link

Unexpected input(s) 'comment' #1186

Open kjetilk-kontali opened 1 month ago

kjetilk-kontali commented 1 month ago

Hi,

I've added the GitHub Action as follows:

- name: Publish Test Report for Integration Tests
  uses: mikepenz/action-junit-report@v4
  if: always()
  with:
    report_paths: '.test-results/*.xml'
    detailed_summary: true
    comment: true
    include_passed: true

But when the action is run, I get the following warning: Warning: Unexpected input(s) 'comment', valid inputs are ['token', 'github_token', 'report_paths', 'test_files_prefix', 'exclude_sources', 'suite_regex', 'annotate_only', 'check_annotations', 'update_check', 'check_name', 'commit', 'fail_on_failure', 'require_tests', 'require_passed_tests', 'include_passed', 'check_title_template', 'summary', 'check_retries', 'transformers', 'job_summary', 'detailed_summary', 'annotate_notice', 'follow_symlink', 'job_name', 'annotations_limit', 'truncate_stack_traces']

I can see from the docs (and the source code) that comment should be a valid input argument to the GH Action, so why am I getting this warning?

mikepenz commented 1 month ago

This feature is a v5 feature and is currently only available in the latest alphas:

mikepenz commented 1 month ago

You can see the @v4 README.md here: https://github.com/mikepenz/action-junit-report/tree/v4

kjetilk-kontali commented 1 month ago

Ah! That figures :)

PS. Might be an idea to mark the input args in the "Inputs" table with the version they're valid in? (e.g. ">5.0")

wmcnamee-coreweave commented 3 weeks ago

just ran into the same thing. Oops! Can I try out v5 some way?

mikepenz commented 3 weeks ago

v5 is available in a03 version: https://github.com/marketplace/actions/junit-report-action?version=v5.0.0-a03

PoqaTJ commented 1 day ago

Just ran into this as well, for both comment and updateComment. The docs should reflect what's available, not what's planned.