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

Allow using a custom name and provide a path to the file that needs to be created using concat. #31

Merged
merged 3 commits into from
Aug 30, 2012

Conversation

vStone
Copy link
Contributor

@vStone vStone commented Aug 30, 2012

In some cases, we want to make the target file dynamic but still be able to add fragments in a generic way. This allows that.

Use case?

We create filenames based on the order files need to be created in (think apache vhosts ordering). Modules still need to be able to use a generic name to add fragments to this file without duplicating the logic in resolving the file path.

@travisbot
Copy link

This pull request passes (merged 1972a9b into 0025977).

@ripienaar
Copy link
Contributor

Looks good, cn you please update the comments to capture the meaning of the argument and I will merge it

@travisbot
Copy link

This pull request passes (merged 6a0c1f9 into 0025977).

@vStone
Copy link
Contributor Author

vStone commented Aug 30, 2012

Dunno how I could have forgotten that being a doc-nazi myself.

ripienaar added a commit that referenced this pull request Aug 30, 2012
Allow using a custom name and provide a path to the file that needs to be created using concat.
@ripienaar ripienaar merged commit 7490f60 into puppetlabs:master Aug 30, 2012
@ripienaar
Copy link
Contributor

thank you :)

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