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

Big update #66

Merged
merged 8 commits into from Mar 29, 2017
Merged

Big update #66

merged 8 commits into from Mar 29, 2017

Conversation

kddnewton
Copy link
Owner

This PR does a couple of things. It:

  1. Drops support for ruby 2.0.x. It's been dropped by ruby for a very long time, and there's no reason to support it anymore. If people need it, they can use an older version of humidifier.
  2. Checks the docs into the docs directory. This greatly simplifies our docs workflow and will hopefully ensure that they always get updated.
  3. Drops the Utils::enumerable_to_h method which was needed to backport the to_h method to ruby 2.0.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7ab1f41 on big-update into 4b28df4 on master.

@@ -1,4 +1,4 @@
module Humidifier
# Gem version
VERSION = '1.0.6'.freeze
VERSION = '1.2.1'.freeze
Copy link
Contributor

Choose a reason for hiding this comment

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

👀 ? why patch .1

Copy link
Owner Author

Choose a reason for hiding this comment

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

To stay in line with the AWS CloudFormation resource specification version (so people know which version is which)

@meagonqz
Copy link
Contributor

What's the process now for updating the docs? Are they still hosted by github on gh-pages?

@kddnewton
Copy link
Owner Author

They'll still be hosted by github pages, but it's just going to be off the master branch docs directory. (This wasn't an option when humidifier was built originally.)

@@ -8,7 +8,7 @@ Humidifier allows you to build AWS CloudFormation (CFN) templates programmatical

For the full docs, go to [https://localytics.github.io/humidifier/](http://localytics.github.io/humidifier/). For local development instructions, see the [Development](https://localytics.github.io/humidifier/#label-Development) section.
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like the Development instructions need to be updated to specify Ruby 2.1 as well if they aren't already (big PR)

https://localytics.github.io/humidifier/#label-Development

Copy link
Contributor

@meagonqz meagonqz left a comment

Choose a reason for hiding this comment

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

🙌 🙌 🙌
Looks great

@coveralls
Copy link

coveralls commented Mar 29, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 9b94024 on big-update into 4b28df4 on master.

@kddnewton kddnewton merged commit b865880 into master Mar 29, 2017
@kddnewton kddnewton deleted the big-update branch March 29, 2017 22:16
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

3 participants