Skip to content

Allow sdk binary in the VMR #48382

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.

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

Merged
merged 3 commits into from
Apr 14, 2025
Merged

Allow sdk binary in the VMR #48382

merged 3 commits into from
Apr 14, 2025

Conversation

Winniexu01
Copy link
Member

@Copilot Copilot bot review requested due to automatic review settings April 11, 2025 05:51
@Winniexu01 Winniexu01 requested review from a team as code owners April 11, 2025 05:51
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Request triage from a team member labels Apr 11, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/SourceBuild/content/eng/allowed-vmr-binaries.txt:121

  • The new allowed binary entry now references a single file without a wildcard pattern and uses a different directory structure compared to the removed entry. Please confirm that this change is intended and that only this specific binary is meant to be allowed; if multiple files should be permitted, consider using a wildcard pattern.
src/sdk/test/dotnet.Tests/ShellShimTests/WpfBinaryTestAssets/testwpf.dll

@mthalman mthalman enabled auto-merge (squash) April 14, 2025 13:07
@mthalman mthalman disabled auto-merge April 14, 2025 19:55
@mthalman mthalman merged commit 9713de6 into dotnet:main Apr 14, 2025
35 of 39 checks passed
@Winniexu01 Winniexu01 deleted the sdkbinary branch April 15, 2025 00:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[main] New sdk binary in VMR
4 participants