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

Issue_749 Add 'managed by puppet' to config files #750

Merged
merged 1 commit into from
Feb 3, 2016

Conversation

alex-harvey-z3q
Copy link
Contributor

No description provided.

@bdronneau
Copy link

👍

@alex-harvey-z3q
Copy link
Contributor Author

If possible, can someone more familiar with the code (& how Nginx works) double check that I didn't miss any files that also should have this message?

@alex-harvey-z3q
Copy link
Contributor Author

Hi @jfryman any chance I could get these PRs I opened reviewed and merged?

@3flex
Copy link
Contributor

3flex commented Feb 2, 2016

Can also add to first line of:

  • templates/conf.d/geo.erb
  • templates/mailhost/mailhost.erb
  • templates/mailhost/mailhost_ssl.erb
  • templates/conf.d/map.erb
  • templates/streamhost/streamhost.erb

Since there's no shared header file between mailhost.erb and mailhost_ssl.erb it's possible that the note will be added twice, but that's not a major concern.

@alex-harvey-z3q
Copy link
Contributor Author

Ok, done.

3flex added a commit that referenced this pull request Feb 3, 2016
Issue_749  Add 'managed by puppet' to config files
@3flex 3flex merged commit 955c685 into voxpupuli:master Feb 3, 2016
Slm0n87 pushed a commit to Slm0n87/puppet-nginx that referenced this pull request Mar 7, 2019
Issue_749  Add 'managed by puppet' to config files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants