My own pet peeve is when folks use `?.let {} ?:` instead of if/else when checking null

It's less readable, and more error prone (because if "doThing()" accidentally returns null, then you end up running the ?: branch, which often is not what you actually want.