Skip to content

Conversation

bryanlatten
Copy link
Contributor

@bryanlatten bryanlatten commented Nov 15, 2016

Can't explicitly test only for specific (and local) nginx site settings, since these tests are also run from downstream clients, who apply their own config.

Also fixed testing volume mounts.

Fixed testing volume mounts
- '/#{0}default_type application/octet-stream;/'
- '/#{0}access_log /dev/stdout main;/'
- '/#{0}default_type\s*application/octet-stream;/'
- '/#{0}access_log\s*/dev/stdout\s*main;/'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whitespace, oh RegEx you rascal.

@mannytoledo
Copy link

👍

@digitalyuki
Copy link

LGTM

@bryanlatten bryanlatten merged commit e693673 into behance:master Nov 15, 2016
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