Zanie Blue
Zanie Blue
I think we'll move _more_ towards Ruff working on files with syntax errors. I'd recommend either excluding these files from lint checks or selecting `E999` to only check for syntax...
Is this an unsafe fix or is this just a bug that we should fix?
Seems reasonable to me. `unnecessary-round-cast` or something?
Could you make this change only apply when preview mode is enabled so we don't increase the scope of the rule without a stabilization period? I think it'd be `checker.settings.preview.is_enabled()`
Here's an example of a deprecated setting: https://github.com/astral-sh/ruff/blob/a73b8c82a83336d76e69ee779d1f0648d1fdc667/crates/ruff_workspace/src/options.rs#L428-L432 which has a warning at https://github.com/astral-sh/ruff/blob/6ed2482e27ec27fa9a6a812cff52efa1c360aded/crates/ruff_workspace/src/configuration.rs#L423-L430 I believe this one was deprecated in https://github.com/astral-sh/ruff/pull/8082 if that's helpful. I'm actually not entirely sure...
Sure. Someone can do this if they want: 1. Drop `Optional` support from `UP007` in preview mode 2. Add new rule in preview for use of `Optional` 3. Add documentation...
Hi! We're going to recategorize all of the rules (roughly discussed in https://github.com/astral-sh/ruff/issues/1774) and define new defaults eventually. We're actively working on this, but I'm not sure when we'll be...
Similarly, this request fits into https://github.com/astral-sh/ruff/issues/1773 which is a part of that effort.
I'm okay with keeping it open for now unless there's a duplicate issue regarding defaults. We should add a dedicated tracking issue for the full re-categorization work eventually.