- Not idempotent. There needs to be a clean target. - () are not good practice in shell variable names (parens are used for execution), use {} to enclose variable names instead - sed with in place editing -i should include a backup file as best practice, and is required when running on a Mac - zsh on Mac does not interpolate $$ without escaping