Skip to content
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

added method to update nuget via Azure DevOps #4

Merged
merged 20 commits into from
Oct 29, 2019
Merged

added method to update nuget via Azure DevOps #4

merged 20 commits into from
Oct 29, 2019

Conversation

tjsullivan1
Copy link
Contributor

@markgar Slight deviation from standard semantic versioning... I think I mostly captured what you were thinking, but feel free to modify the yml pipeline (specifically this line: Version: '$(Major).$(Minor).$(Patch).$(Build)-preview') if you wanted it to be slightly different.

You should be able to get this to work as is by merging in and then configuring a new service connection to Nuget.org with an API Key. You will then need to modify line 39 of azure-pipelines-preview.yml to have the name of the service connection (publishFeedCredentials: 'nuget.org'). If you name it nuget.org, it should just straight up work.

I started looking at github actions for this, and I think it will be doable eventually, but there are currently some bugs that I would like to see worked out first. I've subscribed to this thread to keep my eye on it: NuGet/Home#8580 (comment)

tjsullivan1 and others added 20 commits October 29, 2019 13:13
trying to specify in a different spot
pulling license file into my own repo
Removed github actions test, there seems to be a bug in the existing nuget build samples (link will be added to PR)
Reverted CIS4Azure.Tests.Pester.nuspec to be more similar to existing one in mark repo. Updated schema version, added license, etc.
This should be the initial version of the azure pipeline spec. Instructions on how to add to AzDO will be added to PR.
@tjsullivan1
Copy link
Contributor Author

I added this to the issue comments, and it is showing a reference, but I'm not sure how we actually mark it ahead of time...

@tjsullivan1 tjsullivan1 marked this pull request as ready for review October 29, 2019 21:53
@markgar markgar merged commit 6511c60 into markgar:master Oct 29, 2019
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.

2 participants