Skip to content

Some RunTests fixups - #32111

Merged
BrennanConroy merged 4 commits into
mainfrom
brecon/runtest
Apr 27, 2021
Merged

Some RunTests fixups#32111
BrennanConroy merged 4 commits into
mainfrom
brecon/runtest

Conversation

@BrennanConroy

Copy link
Copy Markdown
Member

Fixed dotnet-dump being installed last for some reason after #31448
And also check if dotnet-dump is installed before trying to use it.

@BrennanConroy BrennanConroy added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Apr 23, 2021
@BrennanConroy BrennanConroy added this to the 6.0-preview5 milestone Apr 23, 2021
@BrennanConroy
BrennanConroy requested a review from a team as a code owner April 23, 2021 18:13

@dougbu dougbu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If this build is green…

Comment thread eng/helix/content/RunTests/TestRunner.cs
Comment thread eng/helix/content/RunTests/ProcessUtil.cs Outdated
Comment thread eng/helix/content/RunTests/TestRunner.cs Outdated
@BrennanConroy
BrennanConroy merged commit a4c6b5d into main Apr 27, 2021
@BrennanConroy
BrennanConroy deleted the brecon/runtest branch April 27, 2021 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants