[workloads] Shorten the workload name even more #74150
Conversation
ghost
added
the
| <!-- These need to be in order - matching rules will try in order. --> | ||
| <ShortNames Include="Microsoft.NET.Workload.Mono.ToolChain.Manifest"> | ||
| <Replacement>MonoToolChainManifest</Replacement> | ||
| <ShortNames Include="Microsoft.NET.Workload.Mono.ToolChain."> |
There was a problem hiding this comment.
Too much shortening on this line? :-)
There was a problem hiding this comment.
nope, it matches from the start ;)
There was a problem hiding this comment.
So we'll end up with mono.manifest or something like that?
There was a problem hiding this comment.
BTW, I noticed there's already a VSIX package in VS with an ID called "Mono", so as long as we have another qualifier we should be okay.
There was a problem hiding this comment.
should end up as MonoManifest-7.0.100
hoyosjs
left a comment
hoyosjs
left a comment
There was a problem hiding this comment.
Sigh... This limit is painful
hoyosjs
dismissed
their stale review
Wait. Looks like it no longer has the Replacement metadata?
|
/backport to release/7.0-rc1 |
|
Started backporting to release/7.0-rc1: https://github.com/dotnet/runtime/actions/runs/2885146213 |
No description provided.