Skip to content
This repository has been archived by the owner on Nov 10, 2017. It is now read-only.

Support for new deployment options #116

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

helaili
Copy link

@helaili helaili commented Sep 13, 2016

As per GitHub's blog and documentation, there are now new options for deployments (transient, production, auto_inactive). This PR implements support for those options.

Note that although the samples in config_file.md don't show entries such as heroku_test_name, those are still carried over in the payload. They would now better fit in the provider_env_name entry.

@brntbeer
Copy link

brntbeer commented Oct 5, 2016

@atmos bumppppp. Any thought about getting this in?

Copy link
Owner

@atmos atmos left a comment

Choose a reason for hiding this comment

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

How are people with older configs supposed to upgrade?

@autoMerge = true
@transientEnvironment = undefined
@productionEnvironment = undefined
@environments = { "production" : {}}
Copy link
Owner

Choose a reason for hiding this comment

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

The change on the environment class from an array to an object seems fishy

@atmos
Copy link
Owner

atmos commented Oct 6, 2016

Can you merge my env_type branch into yours?

@willdurand
Copy link
Contributor

How are people with older configs supposed to upgrade?

BC break should be avoided IMO, so both syntaxes should be supported.

@meltwater-haven
Copy link

any update on this ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants