FantasticFiasco / serilog-sinks-udp

A Serilog sink sending UDP packages over the network.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

NetStandard 1.3 and csproj migration

scottamartin opened this issue · comments

Step 1: Describe your environment

  • Develop on OS X and Windows
  • Run code on Windows and Ubuntu

Step 2: Describe the problem

I would like to migrate the project to the new csproj format and downgrade the NetStandard reference to 1.3 to match Serilog. From what I can tell 1.3 is the lowest this library could use due to needing System.Net.Sockets.

Hi scottamartin!

That would be a fantastic contribution. I assume that also would include migrating solution file to VS2017?

Yes, that is correct. As far as the build script goes, I would would need to update the image for AppVeyor to VS2017 and that would allow for the removal of the "install" portion.

Removing code is my favorite type of implementation, just knock yourself out.

And again, thanks for the time and effort you put into this.

How is the code treating you, are encountering any problems in the migration? If you desire I can team up with you and help you push this code to production.

Everything is going well, I ran into a larger than normal workload this week. The "dotnet migration" tool is not working, so I am manually converting the project to csproj. I am going to try and devote some time to this today and over this weekend.

@FantasticFiasco PR has been submitted, let me know if you see anything. I am able to run the build and tests on my windows machine (cmd line and Visual Studio 2017). I am also able to restore, build, and test on my mac using the "-f netstandard1.3" flag to specify the framework.

If you can build a dev package, I would be glad to test this in my current projects.