Consider:
where you want to share knowledge, or do not have a strong opinion on if this change is necessary in order to get an approval for the pull request.Should:
where the code in question should be changed because it is buggy, has some dangerous side effect or introduces a bad performance hit. These types of comments often spark a discussion, so try to keep it positive and rational. If you cannot back this comment on with meaningful arguments, you should not have made this comment to begin with.hits / (hits + misses + partials)