Improve warnings when ignoring preview rules without preview flag

[bersbersbers]

ruff check --isolated --ignore PLC1901,PLR0904  # version 0.0.290

outputs

warning: Selection of nursery rule `PLC1901` without the `--preview` flag is deprecated.
warning: Selection `PLR0904` has no effect because the `--preview` flag was not included.

I think these warnings are rather irrelevant - they basically confirm what the users wants to do anyway, which is not select these rules.

[silverwind]

Also noticed this strange warning. I had PLC1901 in my ignore config for a long time (because it's a unsafe rule), ruff v0.0.290 now suddenly started warning about it.

[hmvp]

Also noticed this strange warning. I had PLC1901 in my ignore config for a long time (because it's a unsafe rule), ruff v0.0.290 now suddenly started warning about it.

That rule moved to the nursery, probably because its unsafe... We had the same..

[silverwind]

Does nursery mean it's disabled by default now when I select=["PL"]?

[charliermarsh]

@silverwind - Yeah, if you do select=["PL"], PLC1901 will not be included.

[silverwind]

Thanks, I'll remove it then from ignore and I see the warning also makes sense. It should probably explain better what the "nursery" is, the online docs also have no info on this topic.

[charliermarsh]

"Nursery" was changed to "Preview" -- we should probably update the docs to reflect that it used to be called "Nursery".

[silverwind]

I see. Maybe just reword the warning to something like this?

warning: Ignoring preview rule `<id>` has no effect unless `--preview` is specified.

[charliermarsh]

\cc @zanieb - when you're back :)

[zanieb]

"select" is kind of a broad term in this warning, we're referring to the use of a rule selector which can be used with any of the command line options e.g. --select / --ignore. I figured this would be a little ambiguous and we could improve it with more careful messaging in a follow-up — thanks for your feedback!