Skip to content

chore: configure failsafe reportsDirectory - #924

Merged
ricellis merged 1 commit into
mainfrom
s1389-it-reports
Oct 20, 2026
Merged

ricellis merged 1 commit into
mainfrom
s1389-it-reports

Conversation

@ricellis

Copy link
Copy Markdown
Member

PR summary

Configure failsafe reportsDirectory

Fixes: s1389

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the
    Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features) - N/A CI
  • Docs have been added / updated (for bug fixes / features) - N/A CI

PR Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • New tests
  • Build/CI related changes
  • Documentation content changes
  • Other (please describe)

What is the current behavior?

Failsafe 3.6.0 upgrade in #899 changed the default report directory, meaning we stopped collecting the failsafe reports.

What is the new behavior?

Configure a reports directory similarly to 9ba0fb4 did for surefire in #897.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@ricellis
ricellis requested a review from a team as a code owner October 19, 2026 11:03
@ricellis ricellis self-assigned this Oct 19, 2026
@ricellis
ricellis merged commit 02a261d into main Oct 20, 2026
10 checks passed
@ricellis
ricellis deleted the s1389-it-reports branch October 20, 2026 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants