Skip to content

Conversation

@Sarga
Copy link
Contributor

@Sarga Sarga commented Nov 1, 2017

No description provided.

@pleshakov
Copy link
Contributor

@Sarga thx! Looks good!

  • Could you set the WorkerShutdownTimeout with some value here in the templates_test.go? The tests in that file make sure that the templates are executed without any errors.
  • Could you squash your commits into a single one?

@Sarga
Copy link
Contributor Author

Sarga commented Nov 1, 2017

@pleshakov done. Thanks for the advice!

@pleshakov pleshakov merged commit cfddc11 into nginx:master Nov 2, 2017
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