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

(maint) Actually fix conditional statement #51

Merged
merged 1 commit into from
Nov 14, 2016

Conversation

HAIL9000
Copy link
Contributor

The previous commit contained an error. Unwrap 'puppet_version' in
the first conditional statement, it should not be passed into
Gem::Version.new().to_s first.

@haus
Copy link

haus commented Nov 14, 2016

conditoinal => conditional ?

@bmjen
Copy link
Contributor

bmjen commented Nov 14, 2016

@HAIL9000 +1 from me... feel free to merge upon fixing the commit msg pointed out by @haus

The previous commit contained an error. Unwrap 'puppet_version' in
the first conditional statement, it should not be passed into
Gem::Version.new().to_s first.
@HAIL9000 HAIL9000 force-pushed the maint/master/please_let_this_end branch from d75896d to aaa6cd9 Compare November 14, 2016 21:00
@HAIL9000
Copy link
Contributor Author

oops typo corrected @haus thanks for pointing it out

@haus haus changed the title (maint) Actually fix conditoinal statement (maint) Actually fix conditional statement Nov 14, 2016
@HAIL9000
Copy link
Contributor Author

HAIL9000 commented Nov 14, 2016

Oh... thanks for fixing the PR title too! Sorry I missed that part 😓

@bmjen bmjen merged commit 949b326 into puppetlabs:master Nov 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants