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

add missing autoindex parameter in template of server resource #1174

Merged
merged 1 commit into from
Feb 9, 2018

Conversation

joekohlsdorf
Copy link
Contributor

@joekohlsdorf joekohlsdorf commented Jan 28, 2018

The server resource has an autoindex parameter but the templates did not reflect this. There was also no testcase for this parameter.

@bastelfreak
Copy link
Member

Hi @joekohlsdorf, thanks for the PR. Can you squash the commits into one?

@joekohlsdorf
Copy link
Contributor Author

Hi @bastelfreak, I squashed the commits into one.
PS for the future: You can also squash on merge, its a GitHub feature.

@bastelfreak
Copy link
Member

I'm aware of the feature, but unless it's something support urgent we want encourage contributors to do it. Many don't know how it works and we then are happy to provide assistance via slack or via our IRC channel.

@bastelfreak bastelfreak merged commit 7eb71a3 into voxpupuli:master Feb 9, 2018
Rubueno pushed a commit to Rubueno/puppet-nginx that referenced this pull request Oct 19, 2020
add missing autoindex parameter in template of server resource
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