Closed cmcginty closed 3 years ago
It looks like there are tabs being mixed into the file which make it really hard to read. (see the github diff output for example)
I usually use spaces, but if tabs is what you prefer that is fine too. Just better to pick one and not mix them together.
I also commented out the two unused variables to fix shellcheck warning. When you finish this feature you can uncomment them.
Thanks for the attention to detail.
Let's stick to spaces since the whole script has already been fixed to use them.
I'll be more accurate next time :)
It looks like there are tabs being mixed into the file which make it really hard to read. (see the github diff output for example)
I usually use spaces, but if tabs is what you prefer that is fine too. Just better to pick one and not mix them together.
I also commented out the two unused variables to fix shellcheck warning. When you finish this feature you can uncomment them.