Skip to content

Add support for .netStandard 2.0#17

Merged
fschultz merged 3 commits into
fschultz:developfrom
rbrian:NetStandard
May 26, 2019
Merged

Add support for .netStandard 2.0#17
fschultz merged 3 commits into
fschultz:developfrom
rbrian:NetStandard

Conversation

@rbrian
Copy link
Copy Markdown

@rbrian rbrian commented Aug 16, 2018

i'm working on a Xamarin Forms project which requires the use of .netStandard 2.0 in one library. I made changes to support it, mainly removing the http streaming functions, as things like HttpContext and HttpResponse don't exist in .netStandard in the same way they do in .net 2.0. I tried to find workarounds, but wasn't successful so far. Would take major refactoring of the save* methods to accomplish it.

@fschultz fschultz self-assigned this Aug 27, 2018
@fschultz
Copy link
Copy Markdown
Owner

Thank you very much for the pull request! I apologize for the late reply.

I will initially merge this to a local branch and start working on bringing this library fully up to .NET Standard/Core compatibility. I'm aiming for this to be a 4.0 release since as you say it will require alot of refactoring as well as replacing GDI+ functions and will update this thread with the progress.

@fschultz fschultz added this to the 4.0.0 milestone Aug 27, 2018
@fschultz fschultz changed the base branch from master to develop May 26, 2019 05:45
@fschultz fschultz merged commit da9f2c2 into fschultz:develop May 26, 2019
fschultz added a commit that referenced this pull request May 26, 2019
@fschultz
Copy link
Copy Markdown
Owner

Thanks once again for the pull request! The develop branch is now updated to be compatible with .NET 4.x, Core 2.x and Standard 2.x. Once the new version is tested and considered stable it will be merged into master as version 4.0.

@rbrian
Copy link
Copy Markdown
Author

rbrian commented May 26, 2019 via email

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