@OlegValteriswithUkraine Paying attention to commit messages is useful because in code review, you have the opportunity to fix a misleading or incomplete commit message. Here, mods aren't going to fix ones that are misleading. I'm asking what the point of reading them is, generally - why is it integral to making the review, if you can understand the edit without it?
If you are preparing to reject, you should of course read the summary to confirm you didn't miss anything. But for an obviously good edit...why is reading the summary useful?
@RyanM precisely to avoid situations like this one? 99% of the time it's fine, but you can easily miss that 1% by not paying attention. Besides, how hard is it to just glance at the line?
To me, a reviewer of a suggested edit is presented with 2 pieces of data to, well, review: the edit itself and the summary. Therefore, it stands to reason that they should pay attention to both. If they choose not to - it's fine, but if they make a mistake, it's their choice, and it should be considered a "bad review".
Joking aside, I've seen a lot of offensive edit summaries. Although I am not sure I've ever seen one paired with an even relatively useful edit (until today)
as for the rest - I looked through most of the approved ones that do not have a rejection date (forked your query), and they are mostly either already handled, innocuous, or just of the "removed X word" form, thankfully.
It'd be nice if it also updated the Edit summary in suggested edits queue, but ¯\_(ツ)_/¯ it's much less visible at least (not that revisions is overly visible to begin with)
Yeah. Very happy to see that the vast majority of cases were handled appropriately. Many were also caught by Smokey when approved.