Skip to content

Add DO-NOT-SIGN comment for files that should not be signed#41185

Merged
NikolaMilosavljevic merged 1 commit intodotnet:masterfrom
NikolaMilosavljevic:donotsign.comment
Aug 22, 2020
Merged

Add DO-NOT-SIGN comment for files that should not be signed#41185
NikolaMilosavljevic merged 1 commit intodotnet:masterfrom
NikolaMilosavljevic:donotsign.comment

Conversation

@NikolaMilosavljevic
Copy link
Member

Infrastructure change to flag files that are signed, but shouldn't like apphost/comhost.

Fixes: #3748

Uses Arcade infra added in 2019: dotnet/arcade#4058

@NikolaMilosavljevic
Copy link
Member Author

@safern do you know if the failing run is a known issue? It seems to be failing in all recent PRs.

runtime (Libraries Test Run release coreclr Windows_NT x86 Debug)

@danmoseley
Copy link
Member

Yes #41215

@NikolaMilosavljevic NikolaMilosavljevic merged commit 0d19859 into dotnet:master Aug 22, 2020
@NikolaMilosavljevic
Copy link
Member Author

/backport to release/5.0

@github-actions
Copy link
Contributor

Started backporting to release/5.0: https://github.com/dotnet/runtime/actions/runs/220034616

@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
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.

Add validation that apphost/comhost are not signed

5 participants