Update ignore-without-code message

[cdce8p]

Description

Followup to #12194 (comment)
Update message for ignore-without-code to better capture the spirit of the check.

I've changed use to consider as suggested since I also feel this is slightly better. I'm open for other / better ideas.

Another question, do we want to remove the space between different error codes? I feel that might be a tick better.

- ... (consider "type: ignore[arg-type, union-attr]" instead)
+ ... (consider "type: ignore[arg-type,union-attr]" instead)

/CC: @hauntsaninja @PeterJCLaw

[hauntsaninja]

Thanks! Will leave open for a bit in case PeterJClaw has suggestions.

Don't feel strongly about the space. Grepping public repos seem to be mixed with space and without space. With space matches better how you'd style it if it was a Generic.

[cdce8p]

Thanks! Will leave open for a bit in case PeterJClaw has suggestions.

Sounds good!

Don't feel strongly about the space. Grepping public repos seem to be mixed with space and without space. With space matches better how you'd style it if it was a Generic.

I feel like that's actually a reason to suggest without space 😄 Just to make the distinction that it's not actual Python code.

[PeterJCLaw]

Thanks for the ping! (& sorry for not getting back to you sooner)
I don't feel strongly about the message, though happy to see the change 🙂. I think "consider" here works well 👍

IMO with the space is slightly more readable -- the larger gap makes it clearer at a glance that there's two (or more) things being ignored, rather than one with a long name, but I don't feel strongly about this either.

[cdce8p]

Thanks for the ping! (& sorry for not getting back to you sooner) I don't feel strongly about the message, though happy to see the change 🙂. I think "consider" here works well 👍

👍🏻 Let's go with consider then.

IMO with the space is slightly more readable -- the larger gap makes it clearer at a glance that there's two (or more) things being ignored, rather than one with a long name, but I don't feel strongly about this either.

Ok, with space it is.

[github-actions]

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

[hauntsaninja]

Thank you!