Skip to content

Conversation

@marcpopMSFT
Copy link
Member

Let's just remove it and see if the test can pass as I don't think that check is necessary anymore.

We can potentially just remove the 1.1 and 2.0 targeting if this is blocking.

dotnet/installer#15265 (comment)

…e it and see if the test can pass as I don't think that check is necessary anymore.

We can potentially just remove the 1.1 and 2.0 targeting if this is blocking.
@ghost
Copy link

ghost commented Jan 12, 2023

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@nagilson nagilson merged commit deb55b7 into main Jan 12, 2023
@nagilson nagilson deleted the marcpopMSFT-removefailingcall branch January 12, 2023 23:20
ViktorHofer pushed a commit that referenced this pull request Jan 17, 2023
marcpopMSFT pushed a commit that referenced this pull request Jan 20, 2023
marcpopMSFT pushed a commit that referenced this pull request Jan 31, 2023
dsplaisted pushed a commit to dsplaisted/sdk that referenced this pull request Jan 31, 2023
dsplaisted pushed a commit to dsplaisted/sdk that referenced this pull request Jan 31, 2023
marcpopMSFT pushed a commit that referenced this pull request Feb 7, 2023
marcpopMSFT added a commit that referenced this pull request Feb 13, 2023
Port fix for environmentinfo.
Realize that the fix won't work for 11.0
Modify the fix to work better following the idea of the other fix that almost worked
marcpopMSFT added a commit that referenced this pull request Feb 15, 2023
Port fix for environmentinfo.
Realize that the fix won't work for 11.0
Modify the fix to work better following the idea of the other fix that almost worked
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.

3 participants