| For non-critical stylistic or refactoring I prefix with "Consider". > Consider extracting this to a separate function to make this more re-usable. This gives the developer the option to disregard the comment. Essentially I am trusting that they will consider it. It shows that I trust their judgment instead of ordering them. For stuff that is more important but not critical I prefix with "I'd recommend". I'd recommend doing [current approach] [migration statement] [your suggestion]. It will [benefit]. > I'd recommend testing UI components in a similar way that a user would interact with it. It will make your tests more resilient and prevent other developers from breaking your code. > I'd recommend moving this logic into it's own function, it will allow this code to be used by future developers and allow you to test it easier. This both declares the recommendation and explains why it was recommended. For stuff that is critical, I might just write it more directly: > We need to extract this into it's own function so it doesn't lead to/cause X. |