Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

JsonFormatter initializer needs to take an options hash, because that's ... #75

Merged
merged 1 commit into from

3 participants

@svoynow

...how the runner calls it now.

Fix for: #74

@xaviershay
Owner

Nice, has an extra commit on it though. Please remove and I'll merge.

@svoynow

done.

@xaviershay xaviershay merged commit cbfb017 into square:master

1 check passed

Details default The Travis CI build passed
@bf4

Any idea when this was last working in a released gem?

@xaviershay
Owner

aaaaah how about I just release it for you now :) Standby.

@xaviershay
Owner

This fix is in 2.6.1 (just released). Sorry about that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jul 19, 2013
  1. @svoynow

    JsonFormatter initializer needs to take an options hash, because that…

    svoynow authored
    …'s how the runner call it now.
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/cane/json_formatter.rb
View
2  lib/cane/json_formatter.rb
@@ -5,7 +5,7 @@ module Cane
# Computes a machine-readable JSON representation from an array of violations
# computed by the checks.
class JsonFormatter
- def initialize(violations)
+ def initialize(violations, options = {})
@violations = violations
end
Something went wrong with that request. Please try again.