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

UtilsSpecs: Unit test corrected compatibility issues with Klondike NuGet server #997

Merged
merged 2 commits into from
Aug 19, 2015

Conversation

ghost
Copy link

@ghost ghost commented Aug 19, 2015

First commit 71b69f3

  • Use Path.Combine instead of Path.DirectorySeparatorChar.ToString()
  • Fixes error if HOMEPATH=\ (Root directory, result was 'H:' or '//data' on Unix)

Second commit fd394fc

  • WARNING: I am not a F# developer and this code could possibly break alternative NuGet server versions :-)
  • "It works on my machine" using the current master of Klondike (https://github.com/themotleyfool/Klondike)

Daniel Mueller added 2 commits August 19, 2015 15:24
- Use Path.Combine instead of  Path.DirectorySeparatorChar.ToString() 
- Fixes error if HOMEPATH=\ (Root directory, result was 'H:\\' or '//data' on Unix)
…ojects#998

- WARNING: I am not a F# developer and this code could possibly break alternative NuGet server versions :-)
- "It works on my machine" using the current master of Klondike (https://github.com/themotleyfool/Klondike)
@ghost ghost changed the title UtilsSpecs: Unit test corrected UtilsSpecs: Unit test corrected compatibility issues with Klondike NuGet server Aug 19, 2015
forki added a commit that referenced this pull request Aug 19, 2015
UtilsSpecs: Unit test corrected   compatibility issues with Klondike NuGet server
@forki forki merged commit 9e00d0a into fsprojects:master Aug 19, 2015
@forki
Copy link
Member

forki commented Aug 19, 2015

cool thanks.

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.

None yet

1 participant