Skip to content

Conversation

@davfost
Copy link
Contributor

@davfost davfost commented May 31, 2024

Migrating EsrpCodeSigning Task From Version 4 to Version 5

@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 31, 2024
@davfost davfost requested review from a team, jkoritzinsky, lbussell, missymessa and mmitche May 31, 2024 23:23
@am11 am11 added area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels May 31, 2024
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@jkoritzinsky jkoritzinsky requested a review from hoyosjs May 31, 2024 23:47
Copy link
Member

@hoyosjs hoyosjs left a comment

Choose a reason for hiding this comment

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

Any test runs? Thought there's macOS issues with MI. If a test works, lgtm

@hoyosjs
Copy link
Member

hoyosjs commented Jun 1, 2024

Also, how does this work given that we can't guarantee cleanup of macOS agents

Copy link
Member

@dkurepa dkurepa left a comment

Choose a reason for hiding this comment

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

@dkurepa
Copy link
Member

dkurepa commented Jun 24, 2024

@mmitche mmitche merged commit edf4ad4 into dotnet:main Jun 25, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jul 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants