-
Notifications
You must be signed in to change notification settings - Fork 24
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
Any plans for DNX and asp.net core? #12
Comments
Hi Kevin, What's do you mean, what's required to be compatible with DNX? |
Well - while I was waiting, I forked the code and pulled it down to recompile as asp.net core (update1). In short, a few things changed:
Essentially, fork the code, load the C# classes into a new asp.net core class library project, and get it to compile and successfully run tests. |
Is it possible to do so without breaking compatibility for people not using DNX yet? |
Yes - it will compile with either 4.5.1 or 5.0. When I get a little free time, I'll do a formal pull request if you're interested in checking it out. We're using it right now in our asp.net core app and it's working like a charm - thanks for working on it! |
It's good to hear! Thank you for the help, a PR is welcome but we may wait until the official release of Core 1.0 before releasing a version of the kit integrating it. |
.NET Core is now released, any updates on a release of csharp-kit for core? |
Yes, we're planning to do it but we don't have an ETA yet. We're busy on other fronts for now, but eventually we'll tackle this. |
Did Kevin5464 ever do that pull request by any chance? |
Will there be a package compatible with .Net Core? |
I've forked this and have it working in a production environment with no issues. Though my implementation doesn't you use all the possible fragments. Unfortunately, I have no access to the "Les Bonne Chose" repository so can't run any of the tests. I would be uncomfortable submitting a pull request without proof that those tests work. To use this I built the updated package in VS2017 and put the resulting nuGet package and in a <?xml version="1.0" encoding="utf-8"?>
<configuration>
<packageSources>
<add key="AspNetVNext" value="https://www.myget.org/F/aspnetcidev/api/v3/index.json" />
<add key="NuGet" value="https://api.nuget.org/v3/index.json" />
<add key="Local" value="./packages" />
</packageSources>
</configuration> Hope this helps. |
For you information, we plan to update the kit to use the asp.net core. |
.NET Standard is what you want to target, that way the lib will run in both .NET core and the full .NET framework. You can go for 1.6 or 2.0. I wouldn't bother with 1.6 anymore though. |
@DavidDeSloovere see #15. Unfortunately, I lack the knowledge of Travis (and time) to update the CI build. |
If prismic wants to up its marketshare, I'm sure they'll fix that. |
Please have a look at and merge the pull request mentioned above. Working with packages that doesn't support netstandard is awful. For most developers these days I would say. So please fix this now! |
This can now be closed. https://github.com/adaptivewebworks/prismicio-netstandard-kit covers this. |
No description provided.
The text was updated successfully, but these errors were encountered: