Hey, I faced the issue, when my environment (container) had a default shell of /bin/ash, but in the reviewdog command, I needed the bash features.

So I've spent some time trying to make reviewdog use different shell without success until I found the following code:

Problem with this, that it uses shell := "sh" always, for unix. Since I run reviewdog in the container I was able to work around the problem with ln -sf /bin/bash /bin/sh but it doesn't feel right to override the global system link and that would be a problem if I needed to do that not in the container.

What do you think about reading the SHELL variable not only for windows but for unix systems as well?

Something like:

shell := os.Getenv("SHELL") 
if shell == "" {
   shell = "sh"
}