Skip to content
This repository has been archived by the owner on Nov 7, 2019. It is now read-only.

This repo is misleading #45

Closed
gegana opened this issue Mar 29, 2017 · 5 comments
Closed

This repo is misleading #45

gegana opened this issue Mar 29, 2017 · 5 comments

Comments

@gegana
Copy link

gegana commented Mar 29, 2017

It does not support netcoreap1.0 - 1.1, you are still targeting .net framework 4.6.1 hence you should not call this .netcore...

@stajs
Copy link
Owner

stajs commented Mar 29, 2017

.NET Core supports multiple TFMs, and that includes full .NET Framework. In fact, we list netcoreapp1.0 as supported.

@stajs stajs closed this as completed Mar 29, 2017
@jimdeane
Copy link

I've been trying to understand what this means for a couple of days now. My understanding (probably totally wrong) was that i could use VS2017 to develop a DOTNET CORE class library and then test it using another DOTNET CORE Test Project, using the SpecFlow.NetCore NuGet package (and modifications to the app.config).

The behaviour appears conflicting.

The .feature.cs gets generated but the Techtalk.Specflow library is missing.

If I include SpecFlow 2.1.0 the build fails with incompatible .net versions.

Am I simply trying to use this in an impossible way or is there a workaround.

We use Specflow universally and a attempting to start a DOTNET CORE project right now and failing miserable.

Thnaks

@erik-inkapool
Copy link
Contributor

erik-inkapool commented Mar 30, 2017

I believe it may be the case that we support dotnet core projects, but SpecFlow does not. Perhaps my understanding is flawed though. Us supporting dotnet core doesn't matter if SpecFlow doesn't, of course.

@jimdeane
Copy link

I created a separate issue for this to be sure I understand what's going on.

@stajs
Copy link
Owner

stajs commented Mar 30, 2017

@gegana I got caught up on the technicality of .NET Core and TFMs. We support netcoreapp1.0 however @erik-lundgren is right, it's a moot point if SpecFlow does not. I should have instead referred you to #39. I'll update the README to clarify.

@jimdeane I'll follow up on #46.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants