Almenon

Results 424 comments of Almenon

I'm not a maintainer. I forked the repo and verified that the action works correctly in my fork, so it looks like maybe some repo/package/org settings are incorrect in this...

I'm not really clear on the difference in behavior between this and the existing `extra_atlantis_dependencies`. You can do this already with the existing functionality. In `extra_atlantis_dependencies` you can specify a...

Also ignore the CI test error, sorry, https://github.com/transcend-io/terragrunt-atlantis-config/pull/311 will fix it. This PR will need to be rebased once that is merged in.

I don't get it, why is there a terraform file in `account/region/env/example/`? I've never seen intermixing of Terragrunt and terraform in the same folder before. Also the source in `account/region/env/example/example.tf`...

I just merged in #311, can you rebase please? That might fix the failing windows test.

I reviewed the code, it looks good but I'll hold off on approving it until the tests pass. Also if you could update the readme to add the new flag...

Not sure. Based on the diff it looks like `DependsOn` should be a list with three elements, but for some reason it's missing one of the expected elements "depender_on_depender_nested". The...

It's also weird because in your code I don't see anything specific to windows / linux 🤔

Merged, but the release CI process for releasing a new build with this change is broken. Can you review https://github.com/transcend-io/terragrunt-atlantis-config/pull/332 please? Hopefully that will fix it.

v1.18.0 is out with this change now :)