sensu-plugins / sensu-plugin

A framework for writing Sensu plugins & handlers with Ruby.
http://sensuapp.org
MIT License
126 stars 117 forks source link

Update json requirement #173

Closed jkugler closed 7 years ago

jkugler commented 7 years ago

All tests pass, but using simplecov doesn't give me a lot of confidence in the level of code coverage, especially the code that might be parsing and usin JSON.

Update json required version.

Description

Bump json requirement to < 3.0.0

Motivation and Context

The requirement to use json < 2 conflicts with rest-client which can use versions later. Depending on which gets require'd first, ruby will throw an error saying it can't resolve the dependency.

How Has This Been Tested?

I ran rake.

Types of changes

Checklist:

Known Caveats

cwjohnston commented 7 years ago

I built a pre-release gem with this change and it's working as expected in my lab. 👍