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

Reporting argument error #161

Closed
wants to merge 2 commits into from
Closed

Conversation

cpb
Copy link

@cpb cpb commented Jul 4, 2013

Given I have pygments, bcat, and rdiscount installed
When I ran my cucumber features with ARUBA_REPORT_DIR=doc
Then I received an ArgumentError from inside lib/aruba/reporting.rb

I poked around the history of SpawnProcess and Reporting to try and confirm that you dropped the filter_ansi flag support, seems so.

I suppose this'll take more work to make Travis happy... I tried tagging it as @wip for jruby, but i didn't take a close look at how to do that right.

@jarl-dk
Copy link
Member

jarl-dk commented Nov 19, 2013

Hi.

Thank you for the interest...

Can you ellaborate a bit. I am new to pygments and bcat and rdiscount. Is this a feature request or a bug report?

There is currently only a @wip-jruby-java-1.6 tag that is ignored and that is only because there is a bug in java 1.6 (see cucumber.yml). We don't have any ignored scenarios for java 1.7 (which is what is found on Travis)

@os97673
Copy link
Member

os97673 commented Nov 20, 2013

we are not supposed to use rdiscount internally but we should use kramdown instead.

@ghost
Copy link

ghost commented Apr 8, 2015

Is this issue still valid? @jarl-dk Close?

@maxmeyer
Copy link
Member

If we decide to implement reporting as a formatter for cucumber - see #256 - this PR will be obsolete. I also prefer to get rid of pygmentize and use https://github.com/jneen/rouge instead which is pure ruby.

@ghost ghost force-pushed the master branch 2 times, most recently from 4b424bf to bbd9993 Compare July 22, 2015 10:50
@ghost
Copy link

ghost commented Aug 19, 2015

Aruba/Reporting is deprecated and will be removed.

@ghost ghost closed this Aug 19, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants