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

(#5955) Support for /etc/issue and /etc/issue.net #60

Merged
merged 2 commits into from
Dec 23, 2016

Conversation

GeoffWilliams
Copy link
Contributor

Introduces new parameters: issue_template, issue_content, issue_net_template, issue_net_content - to control what if anything is written to /etc/issue and /etc/issue.net respectively. Includes tests and documentation.

Introduces new parameters: issue_template, issue_content, issue_net_template, issue_net_content - to control what if anything is written to /etc/issue and /etc/issue.net respectively.  Includes tests and documentation.
@bmjen
Copy link
Contributor

bmjen commented Dec 16, 2016

Thanks @GeoffWilliams ! This LGTM, but can you do one additional thing? Can you add '/etc/issue' and '/etc/issue.net` to the first line of the README after '/etc/motd'?
https://github.com/puppetlabs/puppetlabs-motd/pull/60/files#diff-04c6e90faac2675aa89e2176d2eec7d8R26

Thanks!

@tphoney tphoney merged commit dadb523 into puppetlabs:master Dec 23, 2016
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