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
replace validate_* calls with datatypes & minor fixes #223
Conversation
9b532c3
to
e7e8279
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Docs look good, but the ToC should still be an h4 for consistency with other module READMEs.
@jbondpdx updated it. |
Awesome, thank you. I get your logic in changing it, but consistency. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docs are OK
manifests/init.pp
Outdated
Pattern[/present|installed|latest|^[.+_0-9a-zA-Z:~-]+$/] $version = 'present', | ||
$package = undef, | ||
Optional[Array] $package_options = undef, | ||
$java_alternative = undef, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't these be Optional[String]
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The goal was to replace all the validate_* calls with datatypes, not to add datatypes to all parameters. I normally mention this, but somehow forgot it here. I added datatypes to all params in init.pp now.
No description provided.