ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

ℹ️ ecosystem check encountered linter errors. (no lint changes; 1 project error)

ruff check --no-cache --exit-zero --ignore RUF9 --output-format concise --preview

warning: The top-level linter settings are deprecated in favour of their counterparts in the `lint` section. Please update the following options in `pyproject.toml`:
  - 'ignore' -> 'lint.ignore'
  - 'select' -> 'lint.select'
  - 'unfixable' -> 'lint.unfixable'
  - 'per-file-ignores' -> 'lint.per-file-ignores'
warning: `PGH001` has been remapped to `S307`.
warning: `PGH002` has been remapped to `G010`.
warning: `PLR1701` has been remapped to `SIM101`.
ruff failed
  Cause: Selection of deprecated rule `E999` is not allowed when preview is enabled.

Let's say you have a very long document and the syntax error is outside the visible area. It might now be unclear to users why the manually triggered action does nothing.

RustRover does show an error when running rustfmt failed because of a syntax error and I find this useful information.

That is interesting. Does it show on every save? I don't mind providing a notification.

That is interesting. Does it show on every save? I don't mind providing a notification.

Only when I issue the Reformat code manually. It doesn't show a notification when the formatting on save fails.

Only when I issue the Reformat code manually. It doesn't show a notification when the formatting on save fails.

I think it's difficult for a language server to know whether something was invoked manually or automatically by the client except for code actions for which there's CodeActionTriggerKind but that's also optional. I'll make this change in a follow-up PR.

I guess RustRover is able to do this reliably because the entire stack is in their control.

I've opened #12176 for now and I can tackle it later.