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

Adding a QuickStart Guide to the NGINX Module #699

Merged
merged 7 commits into from
Oct 11, 2015
Merged

Adding a QuickStart Guide to the NGINX Module #699

merged 7 commits into from
Oct 11, 2015

Conversation

chadothompson
Copy link
Contributor

I've written a bit of documentation regarding the NGINX Puppet module - mostly a 'quick start' guide or people who are either new to this module, or new to NGINX itself. I have a couple of motivations for adding to the documentation:

  1. I have used this module a lot - it's been very helpful in managing the infrastructure I manage day-to-day. I am, of course, grateful for the work the contributors have put in to the module and I also want to encourage use (and reuse) as I've seen people jump into creating hard-to-reuse manifests because they didn't understand this module or how to use it. (Mostly by beginners.)
  2. I've written instructions similar to this a number of times - rather than continue that work, I'd like to contribute this quick guide to the project itself.

I hope you find this useful!

@jfryman
Copy link
Contributor

jfryman commented Oct 11, 2015

This is an absolutely wonderful contribution. Documentation is usually the hardest to do, and first to go when time is an issue.

Many 🙇.

jfryman added a commit that referenced this pull request Oct 11, 2015
Adding a QuickStart Guide to the NGINX Module
@jfryman jfryman merged commit 9863624 into voxpupuli:master Oct 11, 2015
slm0n87 pushed a commit to slm0n87/puppet-nginx that referenced this pull request Mar 7, 2019
Adding a QuickStart Guide to the NGINX Module
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.

2 participants