Is it not?

> Prefer checking exit statuses over output in `if` statements (i.e. `if  grep -q ...; `, not `if [ -n "$(grep ...)" ];`.

Missed closing parent notwithstanding.