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

Support for Multi formatter #57

Merged
merged 5 commits into from
Feb 26, 2016
Merged

Conversation

gkushang
Copy link
Collaborator

options.formats

Supports multiple formatter.
Type: Array
Available: ['pretty', 'progress', 'summary', 'html']

e.g. formats: ['html', 'pretty']

Note: html formatter will provide Json as well as html report. Multiple formatter is supported for cucumber v@0.8.0 or higher

@gkushang
Copy link
Collaborator Author

@mavdi Let me know what you think about this? Its keeping backward-compatibility by adding formats option, whereas format will still work with cucumber v@0.7.0 and lesser.

@gkushang
Copy link
Collaborator Author

Hi @mavdi, can you take a look this PR? Thanks!!

e.g. `formats: ['html', 'pretty']`

Note: `html` formatter will provide Json as well as `html` report. Multiple formatter is supported for cucumber v@0.8.0 or higher

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add a peerDependency in package.json to make sure that cucumber@^0.8.0? This way we make sure that users do have the correct dependencies prior to using this feature.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nikulkarni thanks for reviewing. Multi-formats are not supported for version 0.7.0 and lesser, but still this plugin will still be used. If we add peerDependency for cucumber@^0.8.0 then plugin would throw the warning/error. What do you think?

@nikulkarni
Copy link
Collaborator

Otherwise looks good to me 👍

gkushang added a commit that referenced this pull request Feb 26, 2016
@gkushang gkushang merged commit e1282b5 into mavdi:master Feb 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants