Skip to content

(MODULES-6216) - Fix type3x function in stdlib #861

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

Merged
merged 2 commits into from
Dec 6, 2017

Conversation

pmcmaw
Copy link
Contributor

@pmcmaw pmcmaw commented Dec 5, 2017

This has been introduced with implementing Rubocop: #838

@@ -15,13 +15,13 @@ module Puppet::Parser::Functions
* boolean
EOS
) do |args|
raise(Puppet::ParseError, "type3x(): Wrong number of arguments given (#{args.size} for 1)") if args.empty?
raise(Puppet::ParseError, "type3x(): Wrong number of arguments given (#{args.size} for 1)") if args.empty? || args.size > 1
Copy link
Contributor

Choose a reason for hiding this comment

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

unless args == 1

@tphoney
Copy link
Contributor

tphoney commented Dec 6, 2017

Otherwise looks good !

@pmcmaw pmcmaw force-pushed the fix_type3x branch 3 times, most recently from 838625b to fbb0a0f Compare December 6, 2017 15:33
@willmeek willmeek merged commit 6c5f178 into puppetlabs:master Dec 6, 2017
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