Skip to content

Only show test results if test phase was run#380

Merged
dosaboy merged 1 commit intocanonical:mainfrom
dosaboy:dont-succeed-unless-tested
Apr 24, 2026
Merged

Only show test results if test phase was run#380
dosaboy merged 1 commit intocanonical:mainfrom
dosaboy:dont-succeed-unless-tested

Conversation

@dosaboy
Copy link
Copy Markdown
Member

@dosaboy dosaboy commented Apr 23, 2026

In other works only show results if we do a full run or re-run the test phase.

@dosaboy dosaboy force-pushed the dont-succeed-unless-tested branch from 63eb054 to 3c901a1 Compare April 23, 2026 15:31
@dosaboy dosaboy requested review from nicolasbock and wilkmar April 23, 2026 15:31
Copy link
Copy Markdown
Contributor

@nicolasbock nicolasbock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change looks good, but the bashate linter is failing.

@dosaboy dosaboy force-pushed the dont-succeed-unless-tested branch from 3c901a1 to 72e9dc1 Compare April 23, 2026 15:47
@dosaboy
Copy link
Copy Markdown
Member Author

dosaboy commented Apr 23, 2026

not ready yet

@dosaboy dosaboy marked this pull request as draft April 23, 2026 15:50
@dosaboy dosaboy force-pushed the dont-succeed-unless-tested branch from 72e9dc1 to bd69a59 Compare April 23, 2026 15:51
In other works only show results if we do a full run
or re-run the test phase.

Also ensure a single test target is known when doing a re-run.
@dosaboy dosaboy force-pushed the dont-succeed-unless-tested branch from bd69a59 to a5fef73 Compare April 23, 2026 17:23
@dosaboy dosaboy marked this pull request as ready for review April 23, 2026 19:14
@dosaboy dosaboy added this pull request to the merge queue Apr 24, 2026
Merged via the queue into canonical:main with commit ce13b41 Apr 24, 2026
3 checks passed
@dosaboy dosaboy deleted the dont-succeed-unless-tested branch April 24, 2026 09:30
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