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 for custom comment in sources.list file #311

Merged

Conversation

juniorsysadmin
Copy link

At the moment source.pp simply adds a comment with the name in the $name.list file. This patch allows a custom comment after the 'file generated by puppet' line. If a comment value is not provided, the default comment remains the name.

@daenney
Copy link

daenney commented Jun 10, 2014

Great, thank you, I'll have a look at it later.

Tip for next time, opening a PR with title '...' is unlikely to cause it to get much attention from our part when we're skimming the list op PR's.

@daenney daenney changed the title ... Allow for custom comment in sources.list file Jun 10, 2014
underscorgan pushed a commit that referenced this pull request Jun 19, 2014
…s-list

Allow for custom comment in sources.list file
@underscorgan underscorgan merged commit bcd1a8d into puppetlabs:master Jun 19, 2014
@juniorsysadmin
Copy link
Author

I just remembered - should I submit a new PR for a modified example in the README.md as well?

@daenney
Copy link

daenney commented Jun 20, 2014

If you want to that'd be great. Mention #311 in the commit so it gets linked here too.

juniorsysadmin added a commit to juniorsysadmin/puppetlabs-apt that referenced this pull request Jun 23, 2014
This patch updates the apt::source example to include a comment (see puppetlabs#311)
daenney pushed a commit that referenced this pull request Jun 23, 2014
…omment-parameter

Update apt::source example in README.md for #311
@juniorsysadmin juniorsysadmin deleted the custom-comment-for-sources-list branch December 7, 2014 01:03
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.

None yet

4 participants