-
Notifications
You must be signed in to change notification settings - Fork 66
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
.NET Core Support #9
Comments
when is this realistic to land? we have a need for it but would implement on our own otherwise. |
Hi @vanthome, I think FSharp.Core should work already with .NET core, that's basically what was needed (+project configuration). Unfortunately I cannot test it on Windows, so would be great if you could try that out. |
Yes, I also read that .net core is now F# ready. My Problem is I want to use the project as a dependency in a netcore app. When I try to restore after adding it, I get this:
Then I tried to import net40 to make it happy like this:
But then it silently just not installs ExcelFinancialFunctions So how can the package be made compatible with netcore? |
AFAIK it should be enough to convert this project to be a portable component targeting profile 259 (rather than profile 7). https://github.com/fsprojects/ExcelFinancialFunctions/blob/master/src/ExcelFinancialFunctions/ExcelFinancialFunctions.Portable.fsproj |
Hmm that sounds darn easy. As I'm using VS Code, it's probably better someone with VS is doing this. Also the project needs to be published to Nuget again. |
@vanthome You don't need to use VS to do this - just change the profile number, rebuild and see what happens. |
@dsyme that seemed to help but it failes with:
|
See this PR: #12 |
@vanthome, please try with the latest package (v2.4). |
Unfortunately I get the same error as with the version before:
|
I want to get this project to build in VS2017, will you be willing to accept a Pull Request? It seems you prefer to use paket rather than nuget. How do I get the two to work harmoniously? I am also new to the F# world, and I know you take the cross platform story very seriously. Any other pitfalls you think I should watch out for? |
Is there any update on this issue? |
Please do, that would be great! |
Now runs on .NET Core. Would love to hear any issues or concerns anyone finds. |
Nice! |
Add support for .NET Core
The text was updated successfully, but these errors were encountered: