Add error message to manual_inspect lint

[flip1995]

r? @Jarcho

changelog: none

[Jarcho]

@bors r+

I don't know why anybody would want error messages.

[bors]

📌 Commit 01a6dfa has been approved by Jarcho

It is now in the queue for this repository.

[bors]

⌛ Testing commit 01a6dfa with merge 1aa236d...

[bors]

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Jarcho
Pushing 1aa236d to master...

[RalfJung]

I don't know why anybody would want error messages.

I honestly can't tell if you are joking...
but in case you are not: it is very confusing to have an error that just says "error". It basically says "something is wrong with your code" but then it doesn't tell me what is wrong with my code, which is something between confusing and irritating.

[Jarcho]

I'm not sure if that could have been less serious. I figured the approval in the same comment would be enough of an indicator.

[RalfJung]

Without any non-verbal cues, interpreting text messages on the internet is very tricky, which is why it is customary to indicate jokes textually, such as with a smiley: ";)". Without such an indication, I think you are expecting too much from random people on the internet being able to read your thoughts. The approval in the same message could easily also have meant " 🤷 but the PR also doesn't hurt so whatever, let's just land it".

Communication among strangers is hard even when tons of explicit cues are given. Let's not make it harder than necessary. :)