ruff: lint.ignore → ignore

[DimitriPapadopoulos]

ignore is part of the Top-level Ruff Settings.

I am sorry for missing that in my previous PR.

[abravalheri]

Hi @DimitriPapadopoulos, see my comment in pypa/setuptools#4125 (comment).

[DimitriPapadopoulos]

Most settings should indeed appear under lint or format, for example:

• keep-runtime-typing

  [lint.pyupgrade]
  # Preserve types, even if a file imports `from __future__ import annotations`.
  keep-runtime-typing = true
  

• quote-style

  [format]
  # Prefer single quotes over double quotes.
  quote-style = "single"
  

However, the Top-Level settings are clearly documented to fall outside these categories:

• ignore

  # Skip unused variable rules (`F841`).
  ignore = ["F841"]
  

[abravalheri]

Thanks @DimitriPapadopoulos, that is a very good point and makes all sense.

I am just a bit confused, because these seems to be some mixed signals. In their discussions, the maintainers say they "recommend¹ it going forward, but the settings at the top-level [ruff] section are still respected for backwards compatibility".

I asked the ruff maintainers further clarification about this in astral-sh/ruff#8297 (reply in thread).

Footnotes

1. In the context where they discuss [lint], so my interpretation is that they recommend [lint] going forward. ↩

[abravalheri]

We got a definitive answer from the ruff maintainers: astral-sh/ruff#8297 (reply in thread)