Skip to content

Split XHarness tests into a separate stage#9105

Merged
premun merged 5 commits intodotnet:mainfrom
premun:prvysoky/xharness-tests
Apr 20, 2022
Merged

Split XHarness tests into a separate stage#9105
premun merged 5 commits intodotnet:mainfrom
premun:prvysoky/xharness-tests

Conversation

@premun
Copy link
Copy Markdown
Member

@premun premun commented Apr 20, 2022

  • Splits the tests into a separate stage so it's easier to tell from the build overview which stage failed
  • Build Debug and Release only once but in different jobs to cross validate both (less Helix jobs, same coverage)
  • Fix artifact/stage names

ghost
ghost previously approved these changes Apr 20, 2022
Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

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

I Like how this looks. The split also allows us to only rerun the new stage if there are any hiccups with the mobile devices.

MattGal
MattGal previously approved these changes Apr 20, 2022
@premun premun dismissed stale reviews from MattGal and ghost via 0ef3c2c April 20, 2022 15:55
@premun premun requested review from a user and MattGal April 20, 2022 16:22
@premun premun merged commit 8e2ccde into dotnet:main Apr 20, 2022
@premun premun deleted the prvysoky/xharness-tests branch April 20, 2022 21:26
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