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

[do not merge] Dotnet core upgrade #31

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

luk355
Copy link

@luk355 luk355 commented Apr 23, 2018

Hi @samjudson,

I did a bit of investigation on how much effort is required to migrate your library to .net core. In the pull request, you can see compiling version of multi-targeted library currently supporting:

  • .net standard 2.0 - therefore .net Core 2.0
  • .net framework 4.0
  • .net framework 4.5

The project can be compiled either by VS or by dotnet cli tool executing dotnet build FlickrNet\FlickrNet.csproj.

I am currently missing support for Silverlight, Monotouch and WindowsCE as I dont have these libraries installed on my computer and therefore I cannot compile the solution once I add these targets. However, to add support for these should be as easy as adding platform targets for these frameworks into TargetFrameworks tag in FlickrNet.csproj file (and uncommenting the commented out code).

If we went down this path, we would essentially end up with with one project targeting all the frameworks and one NuGet package.

The other option would be to create project just for .net standard which would result in a code duplication.

What are you thoughts on this?

@luk355
Copy link
Author

luk355 commented Dec 12, 2018

any feedback, @samjudson?

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.

1 participant