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

[.NETCore] extract SpecFlow reports #1036

Closed
SabotageAndi opened this Issue Feb 16, 2018 · 8 comments

Comments

5 participants
@SabotageAndi
Collaborator

SabotageAndi commented Feb 16, 2018

To make the implementation of .NET Standard/Core support easier, we will drop the current reports.
Not every API we use to generate the reports is available on .NET Standard.
The reports have bugs and don't support the latest versions of the test runners.

We also think that the current integration in specflow.exe is sub-optimal, because we can only update the reports together with SpecFlow.

We are looking forward to pull requests so we can add reports back to SpecFlow later.

@gasparnagy

This comment has been minimized.

Collaborator

gasparnagy commented Feb 26, 2018

Instead of dropping it, it would be probably better to move it to a separate repo, so if someone needs it they can still maintain it. But I agree to take it out from the main repo to make the maintenance easier.

@SabotageAndi

This comment has been minimized.

Collaborator

SabotageAndi commented Feb 28, 2018

Ok, dropping is the wrong word for it. I want to have it separate from main SpecFlow. So extract would be better.

I wil create a new repo SpecFlow.Reports and put the current source of https://github.com/techtalk/SpecFlow/tree/master/TechTalk.SpecFlow.Reporting there, before we remove it from the main repo.

@SabotageAndi SabotageAndi referenced this issue Apr 12, 2018

Closed

Reports generator dependencies #1107

8 of 19 tasks complete

@SabotageAndi SabotageAndi moved this from Todo to In Progess in .NET Core Support Apr 24, 2018

@SabotageAndi SabotageAndi referenced this issue Apr 24, 2018

Merged

Remove Reports #1132

1 of 7 tasks complete

@SabotageAndi SabotageAndi changed the title from [.NETCore][RFC] drop SpecFlow reports to [.NETCore] extract SpecFlow reports Apr 26, 2018

@SabotageAndi SabotageAndi referenced this issue Jun 7, 2018

Closed

[.NETCore] dotnet specflow command #845

0 of 3 tasks complete
@SabotageAndi

This comment has been minimized.

Collaborator

SabotageAndi commented Oct 17, 2018

Actual code is now copied to the new repository.

@BasHamer

This comment has been minimized.

BasHamer commented Oct 23, 2018

the breaking changes document still references this ticket (https://github.com/techtalk/SpecFlow/blob/master/docs/Breaking%20Changes%20with%20SpecFlow%203.0.md) as it is closed that is confusing.

@SabotageAndi

This comment has been minimized.

Collaborator

SabotageAndi commented Oct 25, 2018

Ok, perhaps it's not clear formulated in the Breaking changes document.
In this issue, we describe more in detail, why we extracted the reports into a separate repository.
As this is done, we closed this issue.

@BasHamer

This comment has been minimized.

BasHamer commented Oct 26, 2018

in case more people wander in here; new repo is
https://github.com/techtalk/SpecFlow.Reports

@LirazShay

This comment has been minimized.

Contributor

LirazShay commented Oct 28, 2018

You will need to remove / update the documentation accordingly
https://specflow.org/documentation/Reporting/

@Stephen-Mc

This comment has been minimized.

Collaborator

Stephen-Mc commented Oct 29, 2018

You will need to remove / update the documentation accordingly
https://specflow.org/documentation/Reporting/

We will most likely archive the SpecFlow 2 version of the documentation, so that users on older versions can still access this information. It will presumably take a while until most users have migrated. Updating that section of the wiki is definitely on the To-Do list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment