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

Adding note about Puppet 3 support #153

Merged
merged 1 commit into from
Jul 3, 2017

Conversation

petems
Copy link
Member

@petems petems commented Feb 19, 2017

@petems petems force-pushed the add_note_about_puppet_3_eol branch from 7271d17 to 3dab108 Compare May 3, 2017 15:07
README.md Outdated
## Puppet 3 support

Puppet 3 is EOL as-of January 2017. The last release of this module that will
support Puppet 3.X and earlier will be 1.2.5
Copy link
Member

Choose a reason for hiding this comment

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

Suggest removing the exact version and just say that the v1 series will support Puppet3 and v2 will support Puppet4.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, meant to edit this, was planning on bumping this version to 3.0.0 (so it matches the Puppet version 😄)

Releasing is cheap, and we've done a number of changes to the module we did since the 1.2.X version could possibly be breaking changes, so I thought might as well bump it up 👍

@petems petems force-pushed the add_note_about_puppet_3_eol branch 2 times, most recently from 1073060 to 634a23f Compare May 4, 2017 10:21
* Adding deprecation warning
* First steps towards the work in voxpupuli#152
@petems petems force-pushed the add_note_about_puppet_3_eol branch from 634a23f to 363f32d Compare July 3, 2017 21:06
@petems petems merged commit 2c378f4 into voxpupuli:master Jul 3, 2017
cegeka-jenkins pushed a commit to cegeka/puppet-redis that referenced this pull request Feb 16, 2021
* Adding deprecation warning
* First steps towards the work in voxpupuli#152
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