Skip to content

Add environment property when initializing using for_environment#63

Merged
jtroe merged 4 commits intomasterfrom
environment-property
Jul 10, 2014
Merged

Add environment property when initializing using for_environment#63
jtroe merged 4 commits intomasterfrom
environment-property

Conversation

@jtroe
Copy link
Copy Markdown
Contributor

@jtroe jtroe commented Jul 9, 2014

This solves issue #62

Allows users to use which environment the configuration is for. In my case, this is especially useful for building paths which are different for each environment, such as dot-net/{environment}/connection_info.txt

Jason Roebuck added 3 commits July 9, 2014 08:23
@thorncp thorncp added this to the 1.1 milestone Jul 9, 2014
@thorncp
Copy link
Copy Markdown
Contributor

thorncp commented Jul 9, 2014

Looks good to me. Needs changelog entries though.

@jtroe
Copy link
Copy Markdown
Contributor Author

jtroe commented Jul 10, 2014

Changelogged!

@ralreegorganon
Copy link
Copy Markdown
Member

Looks good

@jtroe
Copy link
Copy Markdown
Contributor Author

jtroe commented Jul 10, 2014

Do we want to do another prerelease?

@ralreegorganon
Copy link
Copy Markdown
Member

If you want to cut a new prerelease after it's merged, that makes sense to me.

jtroe added a commit that referenced this pull request Jul 10, 2014
Add environment property when initializing using for_environment
@jtroe jtroe merged commit 9d0c776 into master Jul 10, 2014
@jtroe jtroe deleted the environment-property branch July 10, 2014 17:53
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