Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

add option to log reports to file or custom logTo function on cucumber #2680

Closed
wants to merge 4 commits into from

Conversation

sebsorin
Copy link

@sebsorin sebsorin commented Nov 5, 2015

Add an option on cucumber framework to log reports to a file or a custom loTo function.
This is helpfull to write json reports in a file for use in jenkins (cucumber/cucumber-js#90).

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@sjelin
Copy link
Contributor

sjelin commented Nov 5, 2015

Needs a test

@sjelin sjelin self-assigned this Nov 5, 2015
@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes and removed cla: no labels Nov 7, 2015
@sjelin
Copy link
Contributor

sjelin commented Nov 16, 2015

I know we should have merged this in ages ago, but now we're moving cucumber support into a separate repo/npm module. Please send your pull request to https://github.com/mattfritz/protractor-cucumber-framework

@sjelin sjelin closed this Nov 16, 2015
@skovalyov
Copy link

@sjelin var cucumberFrameworkPath = require('protractor-cucumber-framework').resolve(); fails with TypeError: Object #<Object> has no method 'resolve'. Probably due to my Node v0.10.29. So I'm using https://github.com/substack/node-resolve for now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants