Always require Pull Requests!

I don't care if you allow your devs to approve their own PRs, and indeed that can be a good compromise to allow them to fix typos etc without review.

Obviously bigger changes should be reviewed before being merged and on larger Looker instances you should require review from someone other than the developer.

Self-approval for small changes does add a small amount of friction for developers, but the benefits outweigh this.

Firstly, keeping changes together in PRs is a good reference for the future.

But MOST importantly, if you have a problem, make a mistake, accidentally delete 45 important Explores (a true story 😂), you can *revert* a Pull Request or Merge Request and undo your mistake before too many people notice...

If you've directly deployed changes within Looker it's a *lot* harder to undo any mistakes.

Trust me on this one 😊