Here is a more recent way to look at things (Not saying that the ones from 2 years ago are wrong)
To avoid "Facebook scroller long" answers, it is OK to post multiple answers.
If you want to mention a lot of stuff in your answer (such as coding conventions, variable names, method length, code duplication, use of exceptions, use of public APIs...), it is often more useful to split the answer into multiple answers.