Skip to content

Small cleanup and send aka.ms link prefix for stable#7203

Merged
mmitche merged 1 commit intodotnet:mainfrom
mmitche:cleanup-and-send-aka-ms-for-stable
Apr 7, 2021
Merged

Small cleanup and send aka.ms link prefix for stable#7203
mmitche merged 1 commit intodotnet:mainfrom
mmitche:cleanup-and-send-aka-ms-for-stable

Conversation

@mmitche
Copy link
Member

@mmitche mmitche commented Apr 7, 2021

Stable builds were not sending the aka.ms link prefix. I think this is safe, now that all the blobs are unique, and typically these are internal anyway.
Also, combine NonStableInternalFeeds and NonStablePublicFeeds into one method, as they were basically identical, aside from one tweak around symbols

Stable builds were not sending the aka.ms link prefix. I think this is safe, now that all the blobs are unique, and typically these are internal anyway.
Also, combine NonStableInternalFeeds and NonStablePublicFeeds into one method, as they were basically identical, aside from one tweak around symbols
Copy link
Contributor

@michellemcdaniel michellemcdaniel left a comment

Choose a reason for hiding this comment

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

LGTM

@mmitche mmitche merged commit 86aee6d into dotnet:main Apr 7, 2021
@mmitche mmitche deleted the cleanup-and-send-aka-ms-for-stable branch April 7, 2021 19:41
akoeplinger pushed a commit to akoeplinger/arcade that referenced this pull request Apr 12, 2021
Stable builds were not sending the aka.ms link prefix. I think this is safe, now that all the blobs are unique, and typically these are internal anyway.
Also, combine NonStableInternalFeeds and NonStablePublicFeeds into one method, as they were basically identical, aside from one tweak around symbols
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants