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

dotnetcore support #7

Open
flytzen opened this issue Jul 20, 2016 · 15 comments
Open

dotnetcore support #7

flytzen opened this issue Jul 20, 2016 · 15 comments

Comments

@flytzen
Copy link

flytzen commented Jul 20, 2016

Any thoughts on whether it'd be worth adding dotnetcore support?
It's sort-of separate from adding aspnetcore support.

@uglybugger
Copy link
Collaborator

Has dotnetcore stopped being a moving target yet? ;)

I'm happy for anyone to have a go at it but I'm not personally going to spend any meaningful effort on anything dotnetcore-related until NUnit, ReSharper and TeamCity all support it nicely.

@flytzen
Copy link
Author

flytzen commented Sep 5, 2016

I think there are a few parts to it.

  1. Change it to use project.json and .xproj. This may change how the Nuget packages are built so may impact the publish pipeline, depending on how the publishing to Nuget is currently handled?
  2. Change the core assembly to "netstandard" as well as the full framework. The other projects should be able to remain as they are.
  3. Maybe, possibly, do a plugin for ASP.Net Core, similar to the MVC one, but not sure it's worth it.

I'd be happy to do the code changes for 1 and 2 - it should be pretty straight forward and I've done a fair bit of work in that space recently. But I'm not sure about the impact on the build/publish pipeline so would need a bit of advice on that. How do you publish to to Nuget at the moment?

@flytzen
Copy link
Author

flytzen commented Sep 16, 2016

I've had a bit more of a look into this. I can see a way to do it - but the forthcoming move away from xproj and back to csproj may make it worthwhile to leave it for the moment.

@grokky1
Copy link

grokky1 commented Mar 13, 2017

@flytzen That has happened now. Did you find a way to make it work with Core?

@flytzen
Copy link
Author

flytzen commented Mar 13, 2017

I didn't, but I was just thinking the other day that I should give it a go, so happy to do that.
Would you be okay with an update to VS2017?

@grokky1
Copy link

grokky1 commented Mar 13, 2017

@flytzen I still haven't made the move to VS2017... Guess it's time. 😆

@flytzen
Copy link
Author

flytzen commented Mar 17, 2017

Just FYI; work has started. Looking quite straight forward so far on the general porting.
However, it relies in turn on ThirdDrawer which will also have to be updated in order to provide proper core support (or dependency remove if it's only minor, I'll look).

Only real question is about how you currently pack? I assume a manual Nuget Pack with the version numbers on the command line? If you use an automatic service, please let me know which.
It'll be slightly different with "dotnet pack", but only a little. I'll include some sample build files you can use and adjust to your workflow.

@flytzen
Copy link
Author

flytzen commented Apr 2, 2017

Update (@grokky1) : Everything is now running on .Net core and there is an ASP.Net Core sample app. Current work is here: https://github.com/flytzen/NotDeadYet/tree/DotNetCore.
I had to remove the dependency on ThirdDrawer.

Main outstanding task is to remove the nuspec files and set up a simple package generation script.

There is one small item that stops it being completely done; NUnit doesn't currently work with Core and VS2017. The simplest short-term fix would be to convert the unit tests to XUnit, but not sure how you feel about that, @uglybugger? That should allow the tests to run with both the frameworks as well.

Of course, MS are rather pissing on this parade with this little gem: https://github.com/aspnet/HealthChecks

@grokky1
Copy link

grokky1 commented Apr 2, 2017

Haven't tried in VS2017. But NUnit definitely works with .NET Core.

@flytzen
Copy link
Author

flytzen commented Apr 3, 2017

@grokky1 Yes, I'm talking about this issue with the runner and the new csproj format: nunit/dotnet-test-nunit#108. Sorry for not being clear.

@grokky1
Copy link

grokky1 commented Apr 4, 2017

Wow what a mess... It's not their fault though. Glad I'm still on vs2015 then 😆

They usually get things right though, so it's only a matter of time before they get it working.

@graemechristie
Copy link

Is there any activity on this ? .NEt core is now 2.0 with 2.1 around the corner. Is this project likely to be maintained going forward ?

@flytzen
Copy link
Author

flytzen commented Nov 23, 2017

@graemechristie It appears to be dead, sadly.

@Jaxelr
Copy link

Jaxelr commented Feb 12, 2018

@flytzen not anymore, check Issue #17

@flytzen
Copy link
Author

flytzen commented Feb 14, 2018

@Jaxelr @uglybugger Ah, cool stuff :)
Do you want to review the PR for the migration to .Net Core? It's currently targeting netstandard16 but could easily be changed to netstandard20 if you prefer. Only outstanding thing is NUnit and the build process, the latter is something we'd need to talk about as it depends on how you want to run that.

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

5 participants