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

Jump to bottom

Move HTML reporter to stryker core #1919

Closed
nicojs opened this issue Dec 11, 2019 · 0 comments · Fixed by #2036
Closed

Move HTML reporter to stryker core #1919

nicojs opened this issue Dec 11, 2019 · 0 comments · Fixed by #2036
Assignees
@simondel
Projects
Backlog
Milestone
3.0

Comments

@nicojs
Copy link
Member

nicojs commented Dec 11, 2019 •
edited

Is your maintenance request related to a problem? Please describe.
Ever since mutation-testing-elements we don't have actual logic in the html reporter package anymore. However, people seem to have problems finding it. For larger projects it is much more helpful than the console reporter

Describe the solution you'd like
Let's add the reporter to the standard list of reporters and enable it by default.
@nicojs nicojs added this to the 3.0 milestone Dec 11, 2019
@simondel simondel added this to Planning in Backlog Feb 12, 2020
@simondel simondel moved this from Planning to In Progress in Backlog Feb 12, 2020
@simondel simondel self-assigned this Feb 12, 2020
@simondel simondel moved this from In Progress to Review in Backlog Feb 14, 2020
@simondel simondel mentioned this issue Feb 14, 2020
Merged
Backlog automation moved this from Review to Done Feb 14, 2020
nicojs pushed a commit that referenced this issue Feb 14, 2020
@simondel
feat(HtmlReporter): include the html reporter in the core package and…
09702d9 
… add it to the default reporters (#2036)

Adds the html reporter to the list of default reporters.
The `@stryker-mutator/html-reporter` package is now deprecated and only a stub. There is no need to install this package anymore since the html reporter is bundled with the `@stryker-mutator/core` package.

Fixes #1919 

BREAKING CHANGE: the `html` reporter is now enabled by default. If you don't want to use it, be sure to override the `reporters` configuration option.

```json
{
  "reporters": ["progress", "clear-text"]
}
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@simondel simondel

Labels
None yet
Projects
Backlog
Done
Milestone
3.0
2 participants
@nicojs @simondel