Publish library symbols to dotnetbuilds#12511
Conversation
| (InstallersAndSymbols, FeedForInstallers), | ||
| (TargetFeedContentType.Checksum, FeedForChecksums), | ||
| (InstallersAndSymbols, FeedStagingForInstallers), | ||
| (TargetFeedContentType.Checksum, FeedStagingForChecksums), |
There was a problem hiding this comment.
Should we also change LN 150 to use this one? and the internal feeds to use the internal equivalent? Looks like we didn't change any3.1 ones, but I can't find whether that was intentional.
Also should we change the .NET 5 ones, or are those dead dead? I never remember.
There was a problem hiding this comment.
3.1 and 5.0 never got the updates to pull from dotnetbuilds. I looked at the blazor lines and it turns out the variables aren't even referenced, so I just deleted them.
There was a problem hiding this comment.
This is changing the 3.1 feeds isn't it? Maybe this change is supposed to go here? https://github.com/dotnet/arcade/blob/main/src/Microsoft.DotNet.Build.Tasks.Feed/src/model/PublishingConstants.cs#L955
fa20d73 to
4cd9556
Compare
|
Okay better now. |
Not sure why this didn't switch over to dotnetbuilds a ways back.