Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Conversation

@MattGal
Copy link
Member

@MattGal MattGal commented Feb 16, 2018

@mmitche can you comment on the .groovy change? SHould 7.4 be in outerloop, replace 7.3, or ?

@MattGal MattGal requested a review from mmitche February 16, 2018 19:58
@mmitche
Copy link
Member

mmitche commented Feb 16, 2018

I'm not sure. @danmosemsft may have an opinion. groovy change looks mechanically sound.

@ianhays ianhays added area-Infrastructure-libraries test enhancement Improvements of test source code labels Feb 20, 2018
@ianhays ianhays modified the milestones: Future, 2.1.0 Feb 20, 2018
@MattGal MattGal requested review from ianhays and removed request for mmitche February 20, 2018 18:23
Copy link
Contributor

@ianhays ianhays left a comment

Choose a reason for hiding this comment

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

Looks good, thanks Matt!

@ianhays
Copy link
Contributor

ianhays commented Feb 20, 2018

@MattGal MattGal merged commit ebcc568 into dotnet:master Feb 20, 2018
@MattGal MattGal deleted the add-some-new-linuxes branch February 27, 2018 22:47
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants