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

remove deprecated rubies #102

Merged

Conversation

phumpal
Copy link
Contributor

@phumpal phumpal commented Sep 30, 2019

Pull Request Checklist

Is this in reference to an existing issue?

General

  • Update Changelog following the conventions laid out here

  • Update README with any necessary configuration snippets

  • Binstubs are created if needed

  • RuboCop passes

  • Existing tests pass

New Plugins

  • Tests

  • Add the plugin to the README

  • Does it have a complete header as outlined here

Purpose

Known Compatibility Issues

@phumpal phumpal force-pushed the chore/remove-deprecated-rubies branch from 6ba5ddf to 1dc8d0e Compare September 30, 2019 21:57
@phumpal
Copy link
Contributor Author

phumpal commented Oct 8, 2019

@majormoses ping

@phumpal
Copy link
Contributor Author

phumpal commented Nov 21, 2019

@majormoses PTAL

Copy link
Member

@majormoses majormoses left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@majormoses majormoses merged commit 2b214a1 into sensu-plugins:master Nov 21, 2019
@majormoses
Copy link
Member

@majormoses
Copy link
Member

@phumpal sorry I have been busy at work and otherwise which have made my time available for reviewing plugins limited.

@phumpal
Copy link
Contributor Author

phumpal commented Nov 22, 2019

@majormoses completely understandable. no worries at all.

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.

None yet

2 participants