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

Ruby 2.4 support #29

Merged
merged 2 commits into from
Jul 3, 2017
Merged

Ruby 2.4 support #29

merged 2 commits into from
Jul 3, 2017

Conversation

Evesy
Copy link
Contributor

@Evesy Evesy commented Jul 2, 2017

Pull Request Checklist

RE sensu-plugins/community#27

General

  • Update Changelog following the conventions laid out on Keep A Changelog

  • RuboCop passes

  • Existing tests pass

Purpose

Ruby 2.4 support

Known Compatablity Issues

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.

@Evesy thank you so much for helping, let's just change the verbiage to testing rather than supporting it. Before declaring that we need to do functional runtime testing on it.

CHANGELOG.md Outdated
@@ -4,6 +4,8 @@ This project adheres to [Semantic Versioning](http://semver.org/).
This CHANGELOG follows the format listed at [Keep A Changelog](http://keepachangelog.com/)

## [Unreleased]
### Added
- Ruby 2.4 support
Copy link
Member

Choose a reason for hiding this comment

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

testing not support. It's just testing install time changes, we are gonna have to also test runtime changes before saying we officially support 2.4.

@majormoses majormoses merged commit 1216306 into sensu-plugins:master Jul 3, 2017
@majormoses
Copy link
Member

@Evesy thank you for helping with this!

@Evesy Evesy deleted the 2.4 branch July 3, 2017 17:59
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