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

[Feature] Support for .netcore 2.0 #54

Closed
SergiusZurawski opened this issue Nov 10, 2017 · 9 comments
Closed

[Feature] Support for .netcore 2.0 #54

SergiusZurawski opened this issue Nov 10, 2017 · 9 comments
Assignees
Milestone

Comments

@SergiusZurawski
Copy link

Is there a chance this project will be ported to Dotnet Core 2.0 ?

@elv1s42 elv1s42 self-assigned this Nov 10, 2017
@elv1s42
Copy link
Member

elv1s42 commented Nov 10, 2017

Hi @SergiusZurawski !

Thank you for this issue. I agree that it would be useful.
Putting 'enhancement' label to resolve it in the future.

@Rogitel
Copy link

Rogitel commented Oct 26, 2018

Any update on this? I love this tool, want to continue using it...

@elv1s42
Copy link
Member

elv1s42 commented Oct 26, 2018

Hi @Rogitel !
Sure, I'll try to include this in the release v0.9.5, not in the next one.

Can you please tell me which framework are you using with Ghpr.Core (SpecFlow, MSTest, NUnit)?
We should also update corresponding projects to .netcore 2.0.

Thank you

@elv1s42
Copy link
Member

elv1s42 commented Oct 26, 2018

@SergiusZurawski , if possible, can you also please tell me which framework are you using with Ghpr.Core (SpecFlow, MSTest, NUnit)?

@elv1s42 elv1s42 changed the title Support for .netcore 2.0 [Feature] Support for .netcore 2.0 Oct 26, 2018
@elv1s42 elv1s42 added this to the v0.9.5 milestone Oct 26, 2018
@Rogitel
Copy link

Rogitel commented Oct 26, 2018

That would be great! I've been using NUnit.

@jmemmons
Copy link
Contributor

jmemmons commented Feb 1, 2019

Hey @elv1s42, can I go ahead and do this or have you already started?

@elv1s42
Copy link
Member

elv1s42 commented Feb 7, 2019

Hi @jmemmons,
Sorry for the late response, for some reason I stoped receiving notification emails about the issue.

I didn't start implementation becuase I don't have much time to work on it and this issue is actually quite a big piece of work. There are some open questions around it like "Should we create a separate solution?", "How to run tests for .NET Core project?", "How do we reuse code?", "How to release mutilple versions?" etc

I think we should try re-use existing classes and add try to them as links to .NET Core project.
Also we will have to review AppVeyor file to build, run tests and publish mutliple solutions.

@jmemmons
Copy link
Contributor

jmemmons commented Feb 7, 2019

I went ahead and started porting the code to .NET Core 2.1 in my forked repository to get an idea of what needs to happen.

I created another project called "Ghpr.Core.NetCore" in the same solution and added a new build configuration for .NET 4.7.2 called "Release-NET47". The projects can live together in the same solution which should make development easier.

The only class that gave me issues was ScreenshotHelper which uses a library that's not available in .NET Core (System.Drawing). I also had trouble with the embedded resources but I wasn't able to spend much time with it. I'd suggest the first thing we do is to port the tests over first so we have a base regression suite we can use to verify that code is not broken during this process.

@elv1s42 elv1s42 mentioned this issue Jun 23, 2019
5 tasks
@elv1s42 elv1s42 modified the milestones: v1.1.0, v1.0 Jun 23, 2019
@elv1s42 elv1s42 modified the milestones: v1.0, v0.9.9 Oct 11, 2019
@elv1s42
Copy link
Member

elv1s42 commented Nov 19, 2019

Released.
Please feel free to reopen this one a raise a new issue in case something is not working.

@elv1s42 elv1s42 closed this as completed Nov 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants