Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove an explicit UseDotNet step when .NET 5 would be available on Devops machines #17118

Closed
pakrym opened this issue Nov 20, 2020 · 4 comments · Fixed by #17281
Closed

Remove an explicit UseDotNet step when .NET 5 would be available on Devops machines #17118

pakrym opened this issue Nov 20, 2020 · 4 comments · Fixed by #17281
Assignees
@pakrym
Labels
EngSys This issue is impacting the engineering system.
Milestone
[2021] January

Comments

@pakrym
Copy link
Contributor

pakrym commented Nov 20, 2020

actions/runner-images#1891
@pakrym pakrym added the EngSys This issue is impacting the engineering system. label Nov 20, 2020
@pakrym pakrym added this to the [2020] December milestone Nov 20, 2020
@pakrym pakrym self-assigned this Nov 20, 2020
@pakrym pakrym mentioned this issue Nov 20, 2020
Merged
@jsquire
Copy link
Member

jsquire commented Dec 16, 2020

The associated pull requests have been merged; closing this out. Please feel free to reopen if this is still tracking active work.

@jsquire jsquire closed this as completed Dec 16, 2020
@pakrym pakrym reopened this Dec 16, 2020
@pakrym
Copy link
Contributor Author

pakrym commented Dec 16, 2020

Still needed. #17281 is the real PR.

@jsquire
Copy link
Member

jsquire commented Dec 16, 2020

Sorry about that. Sliding the milestone for compliance.

@pakrym
Copy link
Contributor Author

pakrym commented Dec 16, 2020

No problem at all. Being more aggressive with the cleanup is something I wholeheartedly support.

@pakrym pakrym mentioned this issue Dec 16, 2020
Merged
@ghost ghost closed this as completed in #17281 Jan 4, 2021
ghost pushed a commit that referenced this issue Jan 4, 2021
@pakrym
Don't install NET5 (#17281)
d744401 
Fixes: #17118

.NET 5 is available, by default, on all agents now.

I'll keep the file so we don't have to re-discover all the places we need to install dotnet the next time.
annelo-msft pushed a commit to annelo-msft/azure-sdk-for-net that referenced this issue Feb 17, 2021
@pakrym @annelo-msft
Don't install NET5 (Azure#17281)
8d7b06a 
Fixes: Azure#17118

.NET 5 is available, by default, on all agents now.

I'll keep the file so we don't have to re-discover all the places we need to install dotnet the next time.
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pakrym pakrym

Labels
EngSys This issue is impacting the engineering system.
Projects
None yet
Milestone
[2021] January
Development

Successfully merging a pull request may close this issue.

Don't install NET5 pakrym/azure-sdk-for-net
2 participants
@jsquire @pakrym