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

Avilable plugins as of 4-14-2014 did not work with solr1.4. My plugin ga... #549

Merged
merged 3 commits into from
Apr 27, 2014

Conversation

mac-flanker
Copy link

...thers same metrics as Pete Shima's plugins.

Differences are I used nokogiri and CSS selectors to parse the XML.

Allowing users to dynamically locate metrics and stats I feel work quite well in my plugin.

PS - I know the spacing looks off. I wrote using sublime text and changed the indentation to us 2 spaces as opposed to 4.

… gathers same metrics as Pete Shima's plugins.

Differences are I used nokogiri and CSS selectors to parse the XML.

Allowing users to dynamically locate metrics and stats I feel work quite well in my plugin.
@kalabiyau
Copy link
Contributor

@mac-flanker can you please check Travis status?

@mac-flanker
Copy link
Author

I have changed all of the tabs and spacing to conform to your style guide.
The one thing I didn't change was the if statement. Is it a must to change
the if statement syntax into using a ternary operator?

On Wed, Apr 23, 2014 at 1:02 PM, Artem Chernikov
[email protected]:

@mac-flanker https://github.com/mac-flanker can you please check Travis
status?


Reply to this email directly or view it on GitHubhttps://github.com//pull/549#issuecomment-41207610
.

@kalabiyau
Copy link
Contributor

@mac-flanker thanks for your work. This is merged with upcoming plan to merge two checks into one to be version agnostic.

kalabiyau added a commit that referenced this pull request Apr 27, 2014
Avilable plugins as of 4-14-2014 did not work with solr1.4. My plugin ga...
@kalabiyau kalabiyau merged commit c703a57 into sensu:master Apr 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants