Sitelet https://github.com/fsprojects/Paket/pull/2147
Skip to content

fix #2138 - #2147

Merged
forki merged 3 commits into
fsprojects:masterfrom
drwatson1:issue2138
Feb 17, 2017
Merged

forki merged 3 commits into
fsprojects:masterfrom
drwatson1:issue2138

Conversation

@drwatson1

Copy link
Copy Markdown
Contributor

@forki I had fixed the issue and now it works for me (and for any on-premise TFS 2017 instances I think).
Can you look at it, please? I think it is not very accurate solution. What do you think about it?

@forki

forki commented Feb 17, 2017

Copy link
Copy Markdown
Member

this basically alwas uses nuget2?

@drwatson1

Copy link
Copy Markdown
Contributor Author

Yes

@forki

forki commented Feb 17, 2017

Copy link
Copy Markdown
Member

But this will then break others

@drwatson1

Copy link
Copy Markdown
Contributor Author

I checked that NuGet.Org works too.
The condition in NuGetV2.fs equals true if I use nuget.org. But it equals false for TFS.

@forki

forki commented Feb 17, 2017

Copy link
Copy Markdown
Member

@drwatson1

Copy link
Copy Markdown
Contributor Author

Yes, I think that way too. Should I remove it?

@forki

forki commented Feb 17, 2017 via email

Copy link
Copy Markdown
Member

@drwatson1

Copy link
Copy Markdown
Contributor Author

Done

@forki
forki merged commit 901a1b7 into fsprojects:master Feb 17, 2017
@forki

forki commented Feb 17, 2017

Copy link
Copy Markdown
Member

thx!

@drwatson1
drwatson1 deleted the issue2138 branch February 17, 2017 16:36
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