-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Point manual publishing to public feed #68558
Conversation
@@ -317,8 +317,9 @@ stages: | |||
command: push | |||
packagesToPush: '$(Build.SourcesDirectory)\artifacts\LanguageServer\*.nupkg' | |||
allowPackageConflicts: false | |||
feedsToUse: select |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is documented as the default?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah I don't need specify this value. However CI passed on this commit and I have another PR out to delete this entire step (as soon as dnceng helps me fix signing) - #68504
So going to leave this as-is for expediency
nuGetFeedType: external | ||
publishFeedCredentials: 'DevDiv - VS package feed' | ||
publishFeedCredentials: 'azure-public/vs-impl' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm working on enabling our normal publishing flow for this package (see #68504), but there are some issues around signing. I'm working with dnceng to get those resolved, but in the meantime this should unblock consumption of the packages in vscode-csharp.
Test build - https://dnceng.visualstudio.com/internal/_build/results?buildId=2198741&view=results