layerform

layerform copied to clipboard

## Why this matters Update README to reflect OpenTF name change, and adding link to new domain https://opentofu.org/ ## Solution Simple verbiage change and added hyperlink. PS! Your template for...

oaustegard

General questions

2

Awesome stuff! This seems like a neat approach to bring ownership boundaries into a Terraform codebase 👍 I have some general questions (disclaimer: I've read about Layerform but haven't tried...

foragerr

Do not allow `configure` to remove a definition when a instance depends on it

1

### Reproducing the bug https://github.com/ergomake/layerform/assets/7764293/ef1e34cf-5b6f-4c33-ac1c-355739032917 1. Configure the `local` example: ```bash layerform configure --file examples/local/layerform.json ``` 2. Spawn an instance of `bar` ```bash layerform spawn bar bar_instance ``` 3. Update...

vieiralucas

## Why this matters Resolves #75 ## Solution Validate that all `layers[*].dependencies` exist. ## How to test it Run `make test`. Otherwise, we can also reproduce the issue by doing...

rafiramadhana

Made all short descriptions of commands start with capitalized letters

3

## Why this matters Short description of some commands were starting with lowercase alphabet. So I corrected them. ## Solution Made changes inside cmd/ folder which was containing cobra commands.

sushant102004

`configure` should validate that all `dependecies` exists

2

### Bug If I write a `layerform.json` like this: ```js { "layers": [ { "name": "foo", "files": ["foo.tf"] }, { "name" : "bar", "files": ["bar.tf"], "dependencies": ["foo", "this_layer_does_not_exist"] }, {...

vieiralucas

With new contributors coming in, we should write a proper `CONTRIBUTING.md` file to guide new developers and offer then a good experience. Here are a few things I think we...

lucasfcosta

In the [Getting Started](https://github.com/ergomake/layerform#getting-started) section, it says > Once you have your infrastructure defined as code, you'll create the layer definitions that the CLI will use when spawning instances of...

rafiramadhana

Make all `short` descriptions of commands start with capitalized letters

5

It is currently inconsistent, some commands start with capital letters other not. We should make sure all commands `short` descriptions start with capitalized letters. Current output of `layerform --help` ```console...

vieiralucas

Add a `--auto-approve` flag to `layerform kill`

4

### Problem: When using `layerform` in a script, or CI environment, I want to be able to `kill` layer instances without having to answer `yes` to the "are you sure?"...

vieiralucas