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

Nuget fixes #479

Merged
merged 10 commits into from
May 18, 2021
Merged

Nuget fixes #479

merged 10 commits into from
May 18, 2021

Conversation

talarian1
Copy link
Contributor

@talarian1 talarian1 commented Apr 20, 2021

  • All tests passed. If this feature is not already covered by the tests, I added new tests.

~ Change NuGet protocol from V2 to V3.
~ Improve dependencies source interpretation.

@github-actions
Copy link

github-actions bot commented Apr 20, 2021

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

Copy link
Contributor

@eyalbe4 eyalbe4 left a comment

Choose a reason for hiding this comment

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

Following our recent experience, I suggest we keep supporting NuGet protocol v2 by default, and allow setting the protocol version to v3.
Currently Jenkins is the only user of this API, so I suggest adding a v3 option to the pipelines DSL of Jenkins, so that uses can choose to use v3.

@talarian1 talarian1 force-pushed the master branch 3 times, most recently from fbfd069 to e7d0f72 Compare May 10, 2021 15:11
@talarian1 talarian1 merged commit 319bf9f into jfrog:master May 18, 2021
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