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

Target .NET Standard 1.0. #11

Closed
wants to merge 2 commits into from

Conversation

lookatmike
Copy link

To start with, this pull request isn't intended to be accepted as-is. It would break for anyone using it in a net40 project and possibly your AppVeyor build as well.

Would you consider adding .NET Standard 1.0 as a target so this library can be used in .NET Core projects? It requires no changes to .cs files to work, but multitargeting certainly adds complexity to project/build management (depending on what AppVeyor supports -- I'm not familiar with it).

I'm also happy to simply keep my netstandard fork updated, just wanted to float the idea of multitargeting.

@AndreyTsvetkov
Copy link
Owner

Thanks @lookatmike ,
I'll take a look if it is possible to make it work with AppVeyor.

@lookatmike
Copy link
Author

Thank you @AndreyTsvetkov!

@Hekku2
Copy link

Hekku2 commented Dec 7, 2017

What is the status for this? (And is there something I can do to help?)

@vikramadhav
Copy link

I think we are here on standoff, Can i help?

@Mackiovello
Copy link

If you guys still want a .NET Standard version of this package, I've forked this since the maintainer of this repo no longer seems active: https://github.com/Mackiovello/Mackiovello.Maybe

@AndreyTsvetkov
Copy link
Owner

Well, I'm back and the .NET Standard version is now live and on Nuget.

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.

5 participants