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

Drop reverse and always require Puppet 4+ style and Ruby 2.1 #21

Merged
merged 4 commits into from
Dec 6, 2019

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Nov 28, 2019

Puppet 3 is dead and our manifests should be written in the modern style.

While this will still work with older versions, this lint check is only intended for Puppet 4 and newer Ruby 2.1 is safe to assume.

Gemnasium has been inoperational for some time and can be removed.
While this will still work with older versions, this lint check is only
intended for Puppet 4 and newer Ruby 2.1 is safe to assume.
Puppet 3 is dead and our manifests should be written in the modern
style.
@@ -3,5 +3,5 @@ source 'https://rubygems.org'
gemspec

group :release do
gem 'github_changelog_generator', :require => false, :git => 'https://github.com/github-changelog-generator/github-changelog-generator' if RUBY_VERSION >= '2.2.2'
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Since this is so rarely released and it usually works on small modules I'd like to avoid it. Especially since this is not managed in any way.

@alexjfisher
Copy link
Member

Looks like this repo needs a label sync

@ekohl
Copy link
Member Author

ekohl commented Dec 6, 2019

Synced the labels and added backwards-incompatible.

@ekohl ekohl merged commit 681ac2d into voxpupuli:master Dec 6, 2019
@ekohl ekohl deleted the drop-reverse branch December 6, 2019 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants