flou / brume

Brume: an AWS CloudFormation deployer
MIT License
12 stars 5 forks source link

Add configuration file selector #6

Closed geronimo-iia closed 6 years ago

geronimo-iia commented 7 years ago
flou commented 7 years ago

I am not really a huge fan of using global statements for the configuration. I'll take a look at how this could be implemented without globals.

flou commented 6 years ago

@geronimo-iia I believe all issues should be resolved in the latest version of brume, especially the configuration switch -c/--config with brume.yml as the default.

If not feel free to reply on this PR.

geronimo-iia commented 6 years ago

Hi @flou I will do some test/merge from last version, see you soon

geronimo-iia commented 6 years ago

@flou hi, for swicth config and default brume.yml it's ok

It missing some cfn function as basic service, and i have to check other refacto

geronimo-iia commented 6 years ago

It missing cfn function, stack walker, etc... I'm closing this issue, your last change conflict with my pull request