-
Notifications
You must be signed in to change notification settings - Fork 754
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
[WIP][.NETCore] Removal of Code- Behind- Generation #929
[WIP][.NETCore] Removal of Code- Behind- Generation #929
Conversation
…o DotNetCore_RemoveCodeGeneration
@SabotageAndi What did you do in this PR exactly? It is hard to get it from the changes... |
I integrated your PoC from https://github.com/gasparnagy/SpecFlow.xUnitAdapter into the SpecFlow solution and made changes that where needed to get the scenarios executed without codebehind file. |
Ah good. Did you get it working locally? (There are couple of ideas how to improve the PoC, I guess the best would be to make these improvements here. Or at least we need to sync the changes. |
Getting it to work was not a big problem. The rest around made more headaches. |
To continue forward with this PR, there has to be made some enhancement to the test project setup code (solution file, nuget restore, nuget feed, ...). |
The test project setup code is now open sources in this repo: https://github.com/techtalk/SpecFlow.TestProjectGenerator The API is similar to the existing ones. So swapping it out should be not that big problem. |
@SabotageAndi great! (although the name |
What's the progress on this? Can we get this merged and released? |
closing PR because we won't follow this idea for the first release of .NET Core support. |
#resolves #824