Skip to content

Conversation

@mmitche
Copy link
Member

@mmitche mmitche commented Sep 1, 2022

Only on in release branches

@ghost
Copy link

ghost commented Sep 1, 2022

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.

@ghost ghost assigned mmitche Sep 1, 2022
@mmitche
Copy link
Member Author

mmitche commented Sep 1, 2022

Test build (with tweaked conditional): https://dev.azure.com/dnceng/internal/_build/results?buildId=1981480&view=results

Copy link
Contributor

@carlossanlop carlossanlop left a comment

Choose a reason for hiding this comment

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

I assume the CI would show if this failed?

@ViktorHofer
Copy link
Member

I assume the CI would show if this failed?

No. The runtime-official.yml pipeline doesn't run publicly on PRs. We will see post-merge if this worked but based on these changes, they should work.

@mmitche
Copy link
Member Author

mmitche commented Sep 2, 2022

@mmitche
Copy link
Member Author

mmitche commented Sep 2, 2022

@carlossanlop can you merge this? org JIT is down so I don't have perms right now

@carlossanlop carlossanlop merged commit 734f981 into dotnet:release/7.0 Sep 2, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Oct 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants