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

BREAKING: Drop Puppet 3 support. Replace validate_* with datatypes #536

Merged
merged 7 commits into from
Mar 21, 2017

Conversation

bastelfreak
Copy link
Member

No description provided.

@bastelfreak
Copy link
Member Author

There are a few strings that could also be converted to Integers. Let me know if I should do this as well. The goal of this PR is to get rid of all the validate_* function calls. They are too noisy since Puppet deprecated them.

@binford2k binford2k merged commit f999c74 into voxpupuli:master Mar 21, 2017
@bastelfreak bastelfreak deleted the puppet4 branch March 21, 2017 20:35
@benh57
Copy link

benh57 commented Jun 2, 2017

This module still warns about the validate_re() calls. can you fix those as well?

@benh57
Copy link

benh57 commented Jun 2, 2017

Warning: This method is deprecated, please use the stdlib validate_legacy function, with Pattern[]. There is further documentation for validate_legacy function in the README. at [".../rabbitmq/manifests/init.pp", 85]:

@wyardley wyardley changed the title replace validate_* with datatypes BREAKING: replace validate_* with datatypes Sep 5, 2017
@wyardley wyardley added the enhancement New feature or request label Sep 5, 2017
@alexjfisher alexjfisher changed the title BREAKING: replace validate_* with datatypes BREAKING: Drop Puppet 3 support. Replace validate_* with datatypes Sep 6, 2017
slm0n87 pushed a commit to slm0n87/puppet-rabbitmq that referenced this pull request Mar 7, 2019
	replace validate_* with datatypes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants