search

testdouble / jasmine-rails

A Jasmine runner for rails projects that's got you covered in both the terminal and the browser
http://rubygems.org/gems/jasmine-rails
MIT License
378 stars 154 forks source link

Allow specifying which reporters to use. Fixes #126 #127

Closed shepmaster closed 10 years ago

shepmaster commented 10 years ago

Instead of having a hard-coded set of files for the console reporter, allow the user to define reporters in jasmine.yml. These can be chosen at runtime with the environment variable REPORTERS, a comma-separated list of reporters.

shepmaster commented 10 years ago

@searls Thoughts?

searls commented 10 years ago

Tests pass? :+1:

Could you please add a note in the README?

shepmaster commented 10 years ago

Could you please add a note in the README?

Absolutely. Would it be ok to link to my repo with the JUnit formatter I'm using?

searls commented 10 years ago

Of course

On Mon, May 12, 2014 at 9:28 PM, Jake Goulding notifications@github.com wrote:

Could you please add a note in the README?

Absolutely. Would it be ok to link to my repo with the JUnit formatter I'm using?

Reply to this email directly or view it on GitHub: https://github.com/searls/jasmine-rails/pull/127#issuecomment-42883765

shepmaster commented 10 years ago

Documented! :fireworks:

searls commented 10 years ago

Landed in 0.9.0