-
Notifications
You must be signed in to change notification settings - Fork 209
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update check title and remove icon #144
Conversation
@dorny, is the project still alive? |
updated Junit to lowercase junit. If the reporter is used as "jest-Junit" in a linux build agent, the test runner throws a `module not found` error related to Junit. Setting `--reporters=jest-junit` solves the problem.
@ldaneliukas , @phjardas , @tdabasinskas , I ran into a similar issue as yourselves where there were features that I needed from this project and looked like it wasn't being updated any time soon, so I took the initiative to fork the project myself and build the enhancements into it with a colleague. https://github.com/phoenix-actions/test-reporting/ |
Signed-off-by: Tomer Figenblat <tomer.figenblat@gmail.com>
I had issues with setting --reporters=jest-Junit". I found I needed to use --reporters=jest-junit" instead.
* Fix tests on non us-EN local env Different locale might result in different alphabetical order of tests in report. Tests using snapshot comparison then fails * Fix code style
got@v12.x has a dependency on @sindresorhus/is@v5.x and it uses optional chaining operator "?.". Probably due to issue vercel/ncc#873 this syntax got into our dist/index.js. For some unknown reason it results in error when action is executed in GitHub runner despite the node version is 16.17.
Update all dependencies to latest versions
This fixes an issue with mocha-junit-reporter returning empty root-level testsuite elements.
- Use the ✅ for passed tests - Use the ⚪ for skipped tests
Update to node16 + recent versions of core and exec packages
Fix dorny#199: Use ✅ instead of ✔️ for better cross platform look
…/traverse-7.23.2 Bump @babel/traverse from 7.22.20 to 7.23.2
Add `SwiftXunitParser` class based on `JavaJunitParser` for `swift-xunit` reporter
Update to Node20
Fix build error
Use NodeJS 18 LTS as default runtime
Updates snapshot files for the jest 29
Adding feature and bug template
# Conflicts: # dist/index.js # dist/index.js.map
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [dorny/test-reporter](https://togithub.com/dorny/test-reporter) | action | minor | `v1` -> `v1.8.0` | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>dorny/test-reporter (dorny/test-reporter)</summary> ### [`v1.8.0`](https://togithub.com/dorny/test-reporter/releases/tag/v1.8.0) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.7.0...v1.8.0) #### What's Changed - CHANGELOG.md by [@​j-catania](https://togithub.com/j-catania) in [https://github.com/dorny/test-reporter/pull/297](https://togithub.com/dorny/test-reporter/pull/297) - Bump [@​babel/traverse](https://togithub.com/babel/traverse) from 7.12.1 to 7.23.2 in /reports/jest by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/303](https://togithub.com/dorny/test-reporter/pull/303) - Bump [@​babel/traverse](https://togithub.com/babel/traverse) from 7.22.20 to 7.23.2 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/304](https://togithub.com/dorny/test-reporter/pull/304) - Add `SwiftXunitParser` class based on `JavaJunitParser` for `swift-xunit` reporter by [@​jozefizso](https://togithub.com/jozefizso) in [https://github.com/dorny/test-reporter/pull/317](https://togithub.com/dorny/test-reporter/pull/317) - Update to Node20 by [@​j-catania](https://togithub.com/j-catania) in [https://github.com/dorny/test-reporter/pull/315](https://togithub.com/dorny/test-reporter/pull/315) - Fix build error by [@​dorny](https://togithub.com/dorny) in [https://github.com/dorny/test-reporter/pull/331](https://togithub.com/dorny/test-reporter/pull/331) - Use NodeJS 18 LTS as default runtime by [@​jozefizso](https://togithub.com/jozefizso) in [https://github.com/dorny/test-reporter/pull/332](https://togithub.com/dorny/test-reporter/pull/332) - Update all packages by [@​jozefizso](https://togithub.com/jozefizso) in [https://github.com/dorny/test-reporter/pull/333](https://togithub.com/dorny/test-reporter/pull/333) - Escape <> characters in suite name by [@​wingyplus](https://togithub.com/wingyplus) in [https://github.com/dorny/test-reporter/pull/236](https://togithub.com/dorny/test-reporter/pull/236) - Adding feature and bug template by [@​j-catania](https://togithub.com/j-catania) in [https://github.com/dorny/test-reporter/pull/341](https://togithub.com/dorny/test-reporter/pull/341) - Update check title and remove icon by [@​ldaneliukas](https://togithub.com/ldaneliukas) in [https://github.com/dorny/test-reporter/pull/144](https://togithub.com/dorny/test-reporter/pull/144) - `test-reporter` release v1.8.0 by [@​jozefizso](https://togithub.com/jozefizso) in [https://github.com/dorny/test-reporter/pull/370](https://togithub.com/dorny/test-reporter/pull/370) #### New Contributors - [@​jozefizso](https://togithub.com/jozefizso) made their first contribution in [https://github.com/dorny/test-reporter/pull/317](https://togithub.com/dorny/test-reporter/pull/317) - [@​wingyplus](https://togithub.com/wingyplus) made their first contribution in [https://github.com/dorny/test-reporter/pull/236](https://togithub.com/dorny/test-reporter/pull/236) - [@​ldaneliukas](https://togithub.com/ldaneliukas) made their first contribution in [https://github.com/dorny/test-reporter/pull/144](https://togithub.com/dorny/test-reporter/pull/144) **Full Changelog**: dorny/test-reporter@v1.7.0...v1.8.0 ### [`v1.7.0`](https://togithub.com/dorny/test-reporter/releases/tag/v1.7.0) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.6.0...v1.7.0) #### What's Changed - Fix [#​199](https://togithub.com/dorny/test-reporter/issues/199): Use ✅ instead of ✔️ for better cross platform look by [@​petrdvorak](https://togithub.com/petrdvorak) in [https://github.com/dorny/test-reporter/pull/200](https://togithub.com/dorny/test-reporter/pull/200) - Verify content of dist/ folder matches build output by [@​dorny](https://togithub.com/dorny) in [https://github.com/dorny/test-reporter/pull/207](https://togithub.com/dorny/test-reporter/pull/207) - Gracefully handle empty nested testsuite elements for JUnit. by [@​rvdlaarschot](https://togithub.com/rvdlaarschot) in [https://github.com/dorny/test-reporter/pull/193](https://togithub.com/dorny/test-reporter/pull/193) - Gracefully handle empty failure tags by [@​haudren-woven](https://togithub.com/haudren-woven) in [https://github.com/dorny/test-reporter/pull/213](https://togithub.com/dorny/test-reporter/pull/213) - Fix [#​208](https://togithub.com/dorny/test-reporter/issues/208) - java-junit: show annotations on PR changed files by [@​atsu85](https://togithub.com/atsu85) in [https://github.com/dorny/test-reporter/pull/209](https://togithub.com/dorny/test-reporter/pull/209) - Only report failure if fail-on-error is set by [@​trond-snekvik](https://togithub.com/trond-snekvik) in [https://github.com/dorny/test-reporter/pull/214](https://togithub.com/dorny/test-reporter/pull/214) - Improve clarity on configuring for forkable repos by [@​abelbraaksma](https://togithub.com/abelbraaksma) in [https://github.com/dorny/test-reporter/pull/211](https://togithub.com/dorny/test-reporter/pull/211) - Suppress "Processing test results from" log by [@​vasanthdharmaraj](https://togithub.com/vasanthdharmaraj) in [https://github.com/dorny/test-reporter/pull/179](https://togithub.com/dorny/test-reporter/pull/179) - Skip listing of files if error parsing is disabled by [@​dorny](https://togithub.com/dorny) in [https://github.com/dorny/test-reporter/pull/216](https://togithub.com/dorny/test-reporter/pull/216) - Correct typo in docs by [@​tangowithfoxtrot](https://togithub.com/tangowithfoxtrot) in [https://github.com/dorny/test-reporter/pull/254](https://togithub.com/dorny/test-reporter/pull/254) - update dependencies by [@​j-catania](https://togithub.com/j-catania) in [https://github.com/dorny/test-reporter/pull/269](https://togithub.com/dorny/test-reporter/pull/269) - Add permissions to example yml files by [@​TurnrDev](https://togithub.com/TurnrDev) in [https://github.com/dorny/test-reporter/pull/263](https://togithub.com/dorny/test-reporter/pull/263) - add feature fail-on-empty by [@​gdams](https://togithub.com/gdams) in [https://github.com/dorny/test-reporter/pull/243](https://togithub.com/dorny/test-reporter/pull/243) - Add dependabot configuration by [@​yeikel](https://togithub.com/yeikel) in [https://github.com/dorny/test-reporter/pull/228](https://togithub.com/dorny/test-reporter/pull/228) - Bump ws from 7.3.1 to 7.5.9 in /reports/jest by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/265](https://togithub.com/dorny/test-reporter/pull/265) - Bump actions/checkout from 2 to 4 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/279](https://togithub.com/dorny/test-reporter/pull/279) - Add new output for url url html by [@​luisito666](https://togithub.com/luisito666) in [https://github.com/dorny/test-reporter/pull/242](https://togithub.com/dorny/test-reporter/pull/242) - Update README.md by [@​IanMoroney](https://togithub.com/IanMoroney) in [https://github.com/dorny/test-reporter/pull/158](https://togithub.com/dorny/test-reporter/pull/158) - Update jest-Junit part of Readme by [@​ryancasburn-KAI](https://togithub.com/ryancasburn-KAI) in [https://github.com/dorny/test-reporter/pull/176](https://togithub.com/dorny/test-reporter/pull/176) - fix: default-valued fields are not mandatory by [@​TomerFi](https://togithub.com/TomerFi) in [https://github.com/dorny/test-reporter/pull/172](https://togithub.com/dorny/test-reporter/pull/172) - Bump ansi-regex from 4.1.0 to 4.1.1 in /reports/jest by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/278](https://togithub.com/dorny/test-reporter/pull/278) - Bump decode-uri-component from 0.2.0 to 0.2.2 in /reports/jest by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/276](https://togithub.com/dorny/test-reporter/pull/276) - Bump minimist from 1.2.5 to 1.2.8 in /reports/jest by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/275](https://togithub.com/dorny/test-reporter/pull/275) - Bump qs from 6.5.2 to 6.5.3 in /reports/jest by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/272](https://togithub.com/dorny/test-reporter/pull/272) - Bump json5 from 2.1.3 to 2.2.3 in /reports/jest by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/271](https://togithub.com/dorny/test-reporter/pull/271) - Bump ansi-regex from 3.0.0 to 3.0.1 in /reports/mocha by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/270](https://togithub.com/dorny/test-reporter/pull/270) - declare 'url' and 'url_html' as action outputs by [@​micha-one](https://togithub.com/micha-one) in [https://github.com/dorny/test-reporter/pull/287](https://togithub.com/dorny/test-reporter/pull/287) - Avoid split on undefined by [@​cazou](https://togithub.com/cazou) in [https://github.com/dorny/test-reporter/pull/258](https://togithub.com/dorny/test-reporter/pull/258) #### New Contributors - [@​petrdvorak](https://togithub.com/petrdvorak) made their first contribution in [https://github.com/dorny/test-reporter/pull/200](https://togithub.com/dorny/test-reporter/pull/200) - [@​rvdlaarschot](https://togithub.com/rvdlaarschot) made their first contribution in [https://github.com/dorny/test-reporter/pull/193](https://togithub.com/dorny/test-reporter/pull/193) - [@​haudren-woven](https://togithub.com/haudren-woven) made their first contribution in [https://github.com/dorny/test-reporter/pull/213](https://togithub.com/dorny/test-reporter/pull/213) - [@​atsu85](https://togithub.com/atsu85) made their first contribution in [https://github.com/dorny/test-reporter/pull/209](https://togithub.com/dorny/test-reporter/pull/209) - [@​trond-snekvik](https://togithub.com/trond-snekvik) made their first contribution in [https://github.com/dorny/test-reporter/pull/214](https://togithub.com/dorny/test-reporter/pull/214) - [@​abelbraaksma](https://togithub.com/abelbraaksma) made their first contribution in [https://github.com/dorny/test-reporter/pull/211](https://togithub.com/dorny/test-reporter/pull/211) - [@​vasanthdharmaraj](https://togithub.com/vasanthdharmaraj) made their first contribution in [https://github.com/dorny/test-reporter/pull/179](https://togithub.com/dorny/test-reporter/pull/179) - [@​tangowithfoxtrot](https://togithub.com/tangowithfoxtrot) made their first contribution in [https://github.com/dorny/test-reporter/pull/254](https://togithub.com/dorny/test-reporter/pull/254) - [@​TurnrDev](https://togithub.com/TurnrDev) made their first contribution in [https://github.com/dorny/test-reporter/pull/263](https://togithub.com/dorny/test-reporter/pull/263) - [@​gdams](https://togithub.com/gdams) made their first contribution in [https://github.com/dorny/test-reporter/pull/243](https://togithub.com/dorny/test-reporter/pull/243) - [@​yeikel](https://togithub.com/yeikel) made their first contribution in [https://github.com/dorny/test-reporter/pull/228](https://togithub.com/dorny/test-reporter/pull/228) - [@​luisito666](https://togithub.com/luisito666) made their first contribution in [https://github.com/dorny/test-reporter/pull/242](https://togithub.com/dorny/test-reporter/pull/242) - [@​IanMoroney](https://togithub.com/IanMoroney) made their first contribution in [https://github.com/dorny/test-reporter/pull/158](https://togithub.com/dorny/test-reporter/pull/158) - [@​ryancasburn-KAI](https://togithub.com/ryancasburn-KAI) made their first contribution in [https://github.com/dorny/test-reporter/pull/176](https://togithub.com/dorny/test-reporter/pull/176) - [@​TomerFi](https://togithub.com/TomerFi) made their first contribution in [https://github.com/dorny/test-reporter/pull/172](https://togithub.com/dorny/test-reporter/pull/172) - [@​micha-one](https://togithub.com/micha-one) made their first contribution in [https://github.com/dorny/test-reporter/pull/287](https://togithub.com/dorny/test-reporter/pull/287) - [@​cazou](https://togithub.com/cazou) made their first contribution in [https://github.com/dorny/test-reporter/pull/258](https://togithub.com/dorny/test-reporter/pull/258) **Full Changelog**: dorny/test-reporter@v1.6.0...v1.7.0 ### [`v1.6.0`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v160) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.5.0...v1.6.0) - [Update to node16 + recent versions of core and exec packages](https://togithub.com/dorny/test-reporter/pull/203) - [Update all dependencies to latest versions](https://togithub.com/dorny/test-reporter/pull/186) - [Fix tests on non us-EN local env](https://togithub.com/dorny/test-reporter/pull/185) ### [`v1.5.0`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v150) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.4.3...v1.5.0) - [Add option to convert backslashes in path pattern to forward slashes](https://togithub.com/dorny/test-reporter/pull/128) - [Add option to generate only the summary from processed test results files](https://togithub.com/dorny/test-reporter/pull/123) ### [`v1.4.3`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v143) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.4.2...v1.4.3) - [Patch java-junit to handle missing time field](https://togithub.com/dorny/test-reporter/pull/115) - [Fix dart-json parsing broken by print message](https://togithub.com/dorny/test-reporter/pull/114) ### [`v1.4.2`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v142) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.4.1...v1.4.2) - [Fix dotnet-trx parsing of passed tests with non-empty error info](https://togithub.com/dorny/test-reporter/commit/43d89d5ee509bcef7bd0287aacc0c4a4fb9c1657) ### [`v1.4.1`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v141) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.4.0...v1.4.1) - [Fix dotnet-trx parsing of tests with custom display names](https://togithub.com/dorny/test-reporter/pull/105) ### [`v1.4.0`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v140) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.3.1...v1.4.0) - [Add support for mocha-json](https://togithub.com/dorny/test-reporter/pull/90) - [Use full URL to fix navigation from summary to suite details](https://togithub.com/dorny/test-reporter/pull/89) - [New report rendering with code blocks instead of tables](https://togithub.com/dorny/test-reporter/pull/88) - [Improve test error messages from flutter](https://togithub.com/dorny/test-reporter/pull/87) ### [`v1.3.1`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v131) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.3.0...v1.3.1) - [Fix: parsing of .NET duration string without milliseconds](https://togithub.com/dorny/test-reporter/pull/84) - [Fix: dart-json - remove group name from test case names](https://togithub.com/dorny/test-reporter/pull/85) - [Fix: net-trx parser crashing on missing duration attribute](https://togithub.com/dorny/test-reporter/pull/86) ### [`v1.3.0`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v130) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.2.0...v1.3.0) - [Add support for java-junit](https://togithub.com/dorny/test-reporter/pull/80) - [Fix: Handle test reports with no test cases](https://togithub.com/dorny/test-reporter/pull/70) - [Fix: Reduce number of API calls to get list of files tracked by GitHub](https://togithub.com/dorny/test-reporter/pull/69) ### [`v1.2.0`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v120) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.1.0...v1.2.0) - [Set `listTests` and `listSuites` to lower detail if report is too big](https://togithub.com/dorny/test-reporter/pull/60) ### [`v1.1.0`](https://togithub.com/dorny/test-reporter/blob/HEAD/CHANGELOG.md#v110) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.0.0...v1.1.0) - [Support public repo PR workflow](https://togithub.com/dorny/test-reporter/pull/56) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 10pm every weekday,before 5am every weekday,every weekend" in timezone Europe/Paris, Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/AmadeusITGroup/otter). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMzAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIzMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [actions/cache](https://togithub.com/actions/cache) | action | major | `v3` -> `v4` | | [actions/checkout](https://togithub.com/actions/checkout) | action | major | `v3` -> `v4` | | [actions/download-artifact](https://togithub.com/actions/download-artifact) | action | major | `v3` -> `v4` | | [actions/upload-artifact](https://togithub.com/actions/upload-artifact) | action | major | `v3` -> `v4` | | [amannn/action-semantic-pull-request](https://togithub.com/amannn/action-semantic-pull-request) | action | minor | `v5.4.0` -> `v5.5.2` | | [aws-actions/configure-aws-credentials](https://togithub.com/aws-actions/configure-aws-credentials) | action | major | `v3` -> `v4` | | [dorny/test-reporter](https://togithub.com/dorny/test-reporter) | action | minor | `v1.7.0` -> `v1.9.1` | | ghcr.io/splunk/workflow-engine-base | container | major | `2.0.12` -> `3.0.0` | --- ### Release Notes <details> <summary>actions/cache (actions/cache)</summary> ### [`v4`](https://togithub.com/actions/cache/compare/v3...v4) [Compare Source](https://togithub.com/actions/cache/compare/v3...v4) </details> <details> <summary>actions/checkout (actions/checkout)</summary> ### [`v4`](https://togithub.com/actions/checkout/blob/HEAD/CHANGELOG.md#v400) [Compare Source](https://togithub.com/actions/checkout/compare/v3...v4) - [Support fetching without the --progress option](https://togithub.com/actions/checkout/pull/1067) - [Update to node20](https://togithub.com/actions/checkout/pull/1436) </details> <details> <summary>actions/download-artifact (actions/download-artifact)</summary> ### [`v4`](https://togithub.com/actions/download-artifact/compare/v3...v4) [Compare Source](https://togithub.com/actions/download-artifact/compare/v3...v4) </details> <details> <summary>actions/upload-artifact (actions/upload-artifact)</summary> ### [`v4`](https://togithub.com/actions/upload-artifact/compare/v3...v4) [Compare Source](https://togithub.com/actions/upload-artifact/compare/v3...v4) </details> <details> <summary>amannn/action-semantic-pull-request (amannn/action-semantic-pull-request)</summary> ### [`v5.5.2`](https://togithub.com/amannn/action-semantic-pull-request/releases/tag/v5.5.2) [Compare Source](https://togithub.com/amannn/action-semantic-pull-request/compare/v5.5.1...v5.5.2) ##### Bug Fixes - Bump tar from 6.1.11 to 6.2.1 ([#​262](https://togithub.com/amannn/action-semantic-pull-request/issues/262) by [@​EelcoLos](https://togithub.com/EelcoLos)) ([9a90d5a](https://togithub.com/amannn/action-semantic-pull-request/commit/9a90d5a5ac979326e3bb9272750cdd4f192ce24a)) ### [`v5.5.1`](https://togithub.com/amannn/action-semantic-pull-request/releases/tag/v5.5.1) [Compare Source](https://togithub.com/amannn/action-semantic-pull-request/compare/v5.5.0...v5.5.1) ##### Bug Fixes - Bump ip from 2.0.0 to 2.0.1 ([#​263](https://togithub.com/amannn/action-semantic-pull-request/issues/263) by [@​EelcoLos](https://togithub.com/EelcoLos)) ([5e7e9ac](https://togithub.com/amannn/action-semantic-pull-request/commit/5e7e9acca3ddc6a9d7b640fe1f905c4fff131f4a)) ### [`v5.5.0`](https://togithub.com/amannn/action-semantic-pull-request/releases/tag/v5.5.0) [Compare Source](https://togithub.com/amannn/action-semantic-pull-request/compare/v5.4.0...v5.5.0) ##### Features - Add outputs for `type`, `scope` and `subject` ([#​261](https://togithub.com/amannn/action-semantic-pull-request/issues/261) by [@​bcaurel](https://togithub.com/bcaurel)) ([b05f5f6](https://togithub.com/amannn/action-semantic-pull-request/commit/b05f5f6423ef5cdfc7fdff00c4c10dd9a4f54aff)) </details> <details> <summary>aws-actions/configure-aws-credentials (aws-actions/configure-aws-credentials)</summary> ### [`v4`](https://togithub.com/aws-actions/configure-aws-credentials/releases/tag/v4) [Compare Source](https://togithub.com/aws-actions/configure-aws-credentials/compare/v3...v4) This tag tracks the latest v4.x.x release </details> <details> <summary>dorny/test-reporter (dorny/test-reporter)</summary> ### [`v1.9.1`](https://togithub.com/dorny/test-reporter/releases/tag/v1.9.1) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.9.0...v1.9.1) #### What's Changed - fix: reporting wrong number of tests in Dart by [@​dominicmh](https://togithub.com/dominicmh) in [https://github.com/dorny/test-reporter/pull/426](https://togithub.com/dorny/test-reporter/pull/426) - Fix problematic retransmission of authentication token (alternative solution) by [@​JojOatXGME](https://togithub.com/JojOatXGME) in [https://github.com/dorny/test-reporter/pull/438](https://togithub.com/dorny/test-reporter/pull/438) - Update development dependencies by [@​jozefizso](https://togithub.com/jozefizso) in [https://github.com/dorny/test-reporter/pull/442](https://togithub.com/dorny/test-reporter/pull/442) #### New Contributors - [@​dominicmh](https://togithub.com/dominicmh) made their first contribution in [https://github.com/dorny/test-reporter/pull/426](https://togithub.com/dorny/test-reporter/pull/426) - [@​JojOatXGME](https://togithub.com/JojOatXGME) made their first contribution in [https://github.com/dorny/test-reporter/pull/438](https://togithub.com/dorny/test-reporter/pull/438) **Full Changelog**: dorny/test-reporter@v1.9.0...v1.9.1 ### [`v1.9.0`](https://togithub.com/dorny/test-reporter/releases/tag/v1.9.0) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.8.0...v1.9.0) #### What's Changed ##### Features - Add support for the `rspec-json` format [#​398](https://togithub.com/dorny/test-reporter/issues/398) - Updated packages [#​405](https://togithub.com/dorny/test-reporter/issues/405) #### Contributors - [@​oscarmampel](https://togithub.com/oscarmampel) made their contribution in [#​398](https://togithub.com/dorny/test-reporter/issues/398) **Full Changelog**: dorny/test-reporter@v1.8.0...v1.9.0 ### [`v1.8.0`](https://togithub.com/dorny/test-reporter/releases/tag/v1.8.0) [Compare Source](https://togithub.com/dorny/test-reporter/compare/v1.7.0...v1.8.0) ##### What's Changed - CHANGELOG.md by [@​j-catania](https://togithub.com/j-catania) in [https://github.com/dorny/test-reporter/pull/297](https://togithub.com/dorny/test-reporter/pull/297) - Bump [@​babel/traverse](https://togithub.com/babel/traverse) from 7.12.1 to 7.23.2 in /reports/jest by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/303](https://togithub.com/dorny/test-reporter/pull/303) - Bump [@​babel/traverse](https://togithub.com/babel/traverse) from 7.22.20 to 7.23.2 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/dorny/test-reporter/pull/304](https://togithub.com/dorny/test-reporter/pull/304) - Add `SwiftXunitParser` class based on `JavaJunitParser` for `swift-xunit` reporter by [@​jozefizso](https://togithub.com/jozefizso) in [https://github.com/dorny/test-reporter/pull/317](https://togithub.com/dorny/test-reporter/pull/317) - Update to Node20 by [@​j-catania](https://togithub.com/j-catania) in [https://github.com/dorny/test-reporter/pull/315](https://togithub.com/dorny/test-reporter/pull/315) - Fix build error by [@​dorny](https://togithub.com/dorny) in [https://github.com/dorny/test-reporter/pull/331](https://togithub.com/dorny/test-reporter/pull/331) - Use NodeJS 18 LTS as default runtime by [@​jozefizso](https://togithub.com/jozefizso) in [https://github.com/dorny/test-reporter/pull/332](https://togithub.com/dorny/test-reporter/pull/332) - Update all packages by [@​jozefizso](https://togithub.com/jozefizso) in [https://github.com/dorny/test-reporter/pull/333](https://togithub.com/dorny/test-reporter/pull/333) - Escape <> characters in suite name by [@​wingyplus](https://togithub.com/wingyplus) in [https://github.com/dorny/test-reporter/pull/236](https://togithub.com/dorny/test-reporter/pull/236) - Adding feature and bug template by [@​j-catania](https://togithub.com/j-catania) in [https://github.com/dorny/test-reporter/pull/341](https://togithub.com/dorny/test-reporter/pull/341) - Update check title and remove icon by [@​ldaneliukas](https://togithub.com/ldaneliukas) in [https://github.com/dorny/test-reporter/pull/144](https://togithub.com/dorny/test-reporter/pull/144) - `test-reporter` release v1.8.0 by [@​jozefizso](https://togithub.com/jozefizso) in [https://github.com/dorny/test-reporter/pull/370](https://togithub.com/dorny/test-reporter/pull/370) ##### New Contributors - [@​jozefizso](https://togithub.com/jozefizso) made their first contribution in [https://github.com/dorny/test-reporter/pull/317](https://togithub.com/dorny/test-reporter/pull/317) - [@​wingyplus](https://togithub.com/wingyplus) made their first contribution in [https://github.com/dorny/test-reporter/pull/236](https://togithub.com/dorny/test-reporter/pull/236) - [@​ldaneliukas](https://togithub.com/ldaneliukas) made their first contribution in [https://github.com/dorny/test-reporter/pull/144](https://togithub.com/dorny/test-reporter/pull/144) **Full Changelog**: dorny/test-reporter@v1.7.0...v1.8.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "before 4am on Monday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://togithub.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/splunk/addonfactory-workflow-addon-release). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMTMuMSIsInVwZGF0ZWRJblZlciI6IjM3LjM2My41IiwidGFyZ2V0QnJhbmNoIjoiZGV2ZWxvcCIsImxhYmVscyI6W119--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
It would seem that the ❌ icon is somewhat redundant in most scenarios that we've seen so far. If we look at the whole workflow run, then with or without
fail-on-error
we will still always see a red cross next to the check. This is also true for the checks visible in the Pull Request itself - we already have a ❌ from Github itself. There doesn't seem to be a reason to duplicate this icon with an additional one in the title of the check.In order to make this a little more useful, I'd propose displaying the total number of tests that succeeded, failed, and were skipped in the title instead. This helps us see a short summary of what happened in the pull request checks section and removes the ❌ icon duplication.
Workflow summary
Before:
Pull request
Before:
After: