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

Update to VS2017 + add NETSTANDARD 2.0 target #18

Open
iamcarbon opened this issue Aug 23, 2017 · 10 comments
Open

Update to VS2017 + add NETSTANDARD 2.0 target #18

iamcarbon opened this issue Aug 23, 2017 · 10 comments

Comments

@iamcarbon
Copy link
Collaborator

Are there plans to update the library to target the new csproj format / NETSTANDARD 2.0?

Am up for doing the port if this is a contribution that the project will accept.

  • Jason
@oocx
Copy link
Owner

oocx commented Aug 23, 2017

That would be a great contribution!

I'm currently working on another project in my free time, I won't have time to do the upgrade myself for at least the next several months.

@iamcarbon
Copy link
Collaborator Author

Awesome! getting started now.

@iamcarbon
Copy link
Collaborator Author

Ok, submitted a PR for NETSTANDARD 2.0 support. Going to work on extending the test coverage over the next few weeks.

@iamcarbon iamcarbon reopened this Aug 23, 2017
@iamcarbon
Copy link
Collaborator Author

Just need to patch up the AppVeyor scripts and we should be GTG on the initial migration.

@oocx
Copy link
Owner

oocx commented Aug 23, 2017

Great, thanks!

I just committed the appveyor.yml. I already changed the image to VS2017 and changed the targetframework in the project files from ".netcoreapp2.0" to ".netstandard2.0".

@iamcarbon
Copy link
Collaborator Author

iamcarbon commented Aug 23, 2017

Awesome. All looks well on the CI integration except for versioning.

Do we want to stick with the original format (0.0.0-beta-{build})) or should we begin managing these explicitly?

For instance: the next release could be 0.1.0-beta.

@oocx
Copy link
Owner

oocx commented Aug 24, 2017

Yes, we should change the versioning to 0.1.0, the next release is a major change compared to the previous release that was based on a preview version of .net core.

I added you to the contributors list of the github project, so that you can checkin commits without me (we are in different time zones and I'm on vacation next week, and I don't want to slow you down).

@iamcarbon
Copy link
Collaborator Author

@oocx -- I think the code is in a good place for a 0.1.0-alpha release.

Let me know if you have 10 minutes or so to chat on Skype with the longer term plans for the project. Am down to take a bigger role and help get this to v1.

AM: jason AT carbonmade DOT com on Skype.

@iamcarbon
Copy link
Collaborator Author

@oocx Hope you had a good vacation! Let me know when you have a few to catch up and plan a release!

@oocx
Copy link
Owner

oocx commented Sep 12, 2017

Thanks! Yes, the vacation was great.

It would be great if you wanted to work on this project. Personally, I don't have time to work on this project at the moment. So if you have ideas for improvements and refactorings, go ahead and implement them. When I created this repo, I just wanted to learn more about how Let's Encrypt work and did not actually plan to create and maintain a "real" project. I was surprised that it seemed to be useful to so many people. I'd be happy if you were willing to take over the project and move this project from a learning experiment / prototype to a real production ready tool :).

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

No branches or pull requests

2 participants