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

Remove JSON formatter #1250

Closed
aslakhellesoy opened this issue Oct 31, 2019 · 1 comment · Fixed by #1650
Closed

Remove JSON formatter #1250

aslakhellesoy opened this issue Oct 31, 2019 · 1 comment · Fixed by #1650
Labels
💔 breaking change This will require a major release
Milestone

Comments

@aslakhellesoy
Copy link
Contributor

As described in #1249 the plan is to deprecate the json formatter in version 7. In version 8 we should remove it.

After the removal people can still generate JSON reports with the standalone json formatter. There are prebuilt executables that will convert output from the protobuf formatter to JSON. This is intended to be used only by people who have to use tools that still only support the old JSON format.

@aslakhellesoy aslakhellesoy added the 💔 breaking change This will require a major release label Oct 31, 2019
@aslakhellesoy aslakhellesoy added this to the 8.0.0 milestone Oct 31, 2019
@gkushang
Copy link

I disagree with this change as it will break a lot of existing modules. I strongly feel cucumber should keep in-built support for JSON formatted as there are lot of existing Report Dashboards, HTML Reporters are dependent on it.

The link https:/cucumber/cucumber/tree/master/json-formatter is broken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💔 breaking change This will require a major release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants