Drop a few rules from the preview default set (#23879)
Summary
--
This PR incorporates some feedback from internal discussions on
[Discord] and
from #23203 into the preview default rule set. In particular, we drop:
- [`PERF401`](https://docs.astral.sh/ruff/rules/PERF401)
- [`PERF403`](https://docs.astral.sh/ruff/rules/PERF403)
- [`PLR1714`](https://docs.astral.sh/ruff/rules/PLR1714)
- [`RET504`](https://docs.astral.sh/ruff/rules/RET504)
- [`TRY300`](https://docs.astral.sh/ruff/rules/TRY300)
A couple of these disagree a bit with Clippy's own categorization. For
example, RET504 is
[let_and_return], which is a Style lint for Clippy. But if users find
it annoying in Python, it makes sense to me to bump it down to Pedantic.
PERF401 and PERF403 also have open issues (#21890, #21891), so I think
it makes
sense to consider them lower severity, at least until those are
resolved.
TRY300 and PLR1714 are just a bit more pedantic than expected and some
people
prefer the styles they flag.
[Discord]:
https://discord.com/channels/1039017663004942429/1082324250112823306/1478345981916348526
[let_and_return]:
https://rust-lang.github.io/rust-clippy/master/?search=assign#let_and_return
Test Plan
--
Future default users
