Financial-Times / n-gage

Shared developer and build tools for FT.com applications and components
9 stars 3 forks source link

make rm not error instead of suppressing its error #206

Closed apaleslimghost closed 5 years ago

apaleslimghost commented 5 years ago

otherwise, it prints rm: cannot remove '.review-app': No such file or directory, which looks like an error when you're scanning CI logs but it isn't one, which is misleading when the build breaks because it's never because of that but it looks like it's an error and it's the first thing you see

closes #187 if done with financial-times/n-heroku-tools#577