Skip to content

Conversation

@vStone
Copy link
Contributor

@vStone vStone commented Oct 4, 2017

Allows for more compact, clean code.

@vStone vStone force-pushed the feature/new_function_ifelse branch from 527c5ec to 001f213 Compare October 4, 2017 06:03
@vStone vStone changed the title Add a new function ifelse to match ruby's tenary operator MODULES-5679: Add a new function ifelse to match ruby's tenary operator Oct 4, 2017
@vStone vStone changed the title MODULES-5679: Add a new function ifelse to match ruby's tenary operator (MODULES-5679) Add a new function ifelse to match ruby's tenary operator Oct 4, 2017
@david22swan
Copy link
Member

LGTM

@david22swan david22swan merged commit 8eb4a64 into puppetlabs:master Oct 27, 2017
@hlindberg
Copy link
Contributor

Why on earth do we need a function like this in stdlib?!? This is written like this in the language if $val { $iftrue } else { $iffalse }.
I understand the motivation (somwhat), but please, do this in a separate module.

@david22swan
Copy link
Member

I apologize for the inconvenience but it has bee asked that I undo this merge, as stated above if you wish to keep this change please do so in a separate module.
Once again I apologize.

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.

4 participants