Only include environment in server name if set. #4

Closed
wants to merge 1 commit into
from

Projects

None yet

1 participant

@MarkBennett
Contributor

This prevents server names from ending with an underscore and blank
environment.

@MarkBennett MarkBennett Only include environment in server name if set.
This prevents server names from ending with an underscore and blank
environment.
ce0390b
@MarkBennett
Contributor

Given our conversation this morning, does it make sense to allow pave to run without an environment? Should it default to dev instead? Alternatively, should it just error out if no env is set?

We should make this behaviour consistent between plow and pave.

I'm OK with closing this PR if it doesn't make sense.

@MarkBennett
Contributor

This doesn't make sense anymore since we're always setting the environment. Closing.

@MarkBennett MarkBennett closed this Apr 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment