Shell

Shell

Made by DeepSource

Empty then clause detected SH-1048

Bug risk
Major

Problematic code:

if [ -e foo ]
then
  # TODO: handle this
fi

Preferred code:

if [ -e foo ]
then
  # TODO: handle this
  true
fi

# Or use the no-op colon operator ":"
if [ -e foo ]
then
  # TODO: handle this
  :
fi

Shells do not allow empty then clauses. They need at least one command (and comments are not commands). If you want a then clause that does nothing, use a dummy command like true.