Yoriyasu Yano

Ah looks like the build failed for `TestIncludeDirsStrict` (in `test` folder). Here are the test logs: ``` %~> go test -v -run TestIncludeDirsStrict$ . [0] === RUN TestIncludeDirsStrict === PAUSE...

Hmm I think the error message is actually correct. I would want terragrunt to give me a non-zero exit code when it finds no terragrunt.hcl config that it should run...

Personally, I think the CI system or script should handle the case where there are no directories and not let terragrunt figure it out. That's a really simple array count...

Ah you got me. I missed that, and if @brikis98 made that decision, then I'm fine going forward with this. I would still argue that this should be handled early...

You can't recursively introspect a `dependency` in a `dependency` block. Note that the way you wrote it also has a chicken-and-egg problem. The `skip` flag means skip pulling the dependency,...

I am generally against using commandline arguments for targeting resources to deploy in a routine scenario, primarily because it is so easy to use `Ctrl+R` or `up arrow` in the...

We actually have something similar we rolled out internally, so would be awesome if `allstar` implements this since that means we can retire that app! With that said, I do...

> I've heard that in some scenarios, the Login of a user can be spoofed. I've not investigated this further, but I wonder if this is a problem for your...

The login of a commit is derived from the Name and Email settings of the commit, so all you have to do is set the Name + Email of the...

For reference, the proposal I am making is to move the merge operation to within the if block here: https://github.com/ossf/allstar/blob/8cc89a1/pkg/policies/branch/branch.go#L644-L647