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

Ensure release allows empty strings #681

Merged
merged 1 commit into from
May 30, 2017

Conversation

HelenCampbell
Copy link

@HelenCampbell HelenCampbell commented May 26, 2017

Ensures $release now accepts empty strings as a value without failing and addition of test

@HelenCampbell HelenCampbell changed the title Test to ensure release allows empty strings (WIP) Test to ensure release allows empty strings May 26, 2017
@HelenCampbell HelenCampbell force-pushed the emptystringing branch 2 times, most recently from e7280cb to 29d39cc Compare May 26, 2017 16:21
@HelenCampbell HelenCampbell changed the title (WIP) Test to ensure release allows empty strings (WIP) Ensure release allows empty strings May 26, 2017
@HelenCampbell HelenCampbell force-pushed the emptystringing branch 2 times, most recently from 8c65cff to 91fcef7 Compare May 26, 2017 16:37
@HelenCampbell HelenCampbell changed the title (WIP) Ensure release allows empty strings Ensure release allows empty strings May 26, 2017
@HelenCampbell HelenCampbell force-pushed the emptystringing branch 2 times, most recently from 90f60e4 to 3128fa7 Compare May 30, 2017 10:56
Test added to make sure when release is set to an empty string the
source file is rendered correctly.
@eputnam
Copy link

eputnam commented May 30, 2017

yay structured facts!

@eputnam eputnam merged commit 61f4e6a into puppetlabs:master May 30, 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.

4 participants