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.

Sign up for GitHub

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

Jump to bottom

Conform to multijson 1.3.1: MultiJson.decode is now MultiJson.load #121

Closed
wants to merge 1 commit into from
Closed

Conform to multijson 1.3.1: MultiJson.decode is now MultiJson.load #121

wants to merge 1 commit into from

Conversation

ronen
Copy link

@ronen ronen commented Apr 16, 2012

i updated to the latest multijson and saw this deprecation message:

.../simplecov-0.6.1/lib/simplecov/result_merger.rb:18:in `resultset': [DEPRECATION] MultiJson.decode is deprecated and will be removed in the next major version. Use MultiJson.load instead.

so made the change

@sferik
Copy link
Collaborator

sferik commented Apr 18, 2012

@ronen You missed a couple spots. I've opened #122 which is a more complete version of the same thing.

@ronen ronen closed this Apr 18, 2012
@ronen
Copy link
Author

ronen commented Apr 18, 2012

@sferik Yes I just did the minimal brain-dead thing, and yes I saw yours which is better. Will close mine out.

@colszowka
Copy link
Collaborator

Thanks for your effort! New version 0.6.2 out in a bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ronen @sferik @colszowka