Skip to content

Conversation

HaoK
Copy link
Member

@HaoK HaoK commented Jul 30, 2020

Also adds env printout to test runner

@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Jul 30, 2020
@HaoK HaoK changed the title Print out helix env variables into logs Remove debian 8 from helix runs Jul 31, 2020
@HaoK HaoK requested a review from a team July 31, 2020 17:11
Copy link
Contributor

@dougbu dougbu left a comment

Choose a reason for hiding this comment

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

Looks fine other than the environment dump

@HaoK HaoK marked this pull request as ready for review July 31, 2020 18:57
@HaoK
Copy link
Member Author

HaoK commented Jul 31, 2020

Fair enough, removed env dump

@ghost
Copy link

ghost commented Jul 31, 2020

Hello @HaoK!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit e6b9a3d into master Jul 31, 2020
@ghost ghost deleted the helix/env branch July 31, 2020 20:35
HaoK added a commit that referenced this pull request Aug 7, 2020
* Add display environment

* Print out env variables as part of helix runs

* Update TestRunner.cs

* Remove Debian 8 from helix matrix

* Remove Debian 8 skip

* Remove debian 8 skip

* Remove debian 8

* Remove debian 8 skip

* Remove Debian 8 skip

* Remove debian 8

* Update Program.cs

* Update TestRunner.cs

* Update TestRunner.cs
This pull request was closed.
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.

3 participants