Skip to content

Conversation

@DimitriPapadopoulos
Copy link
Contributor

Subject: Use (new) ruff config section lint

Feature or Bugfix

  • Refactoring

Purpose

  • Adapt to ruff 0.1.9

Detail

As suggested by Repo-Review:

  • RF202: Use (new) lint config section
    flake8-quotes should be set as lint.flake8-quotes instead
    per-file-ignores should be set as lint.per-file-ignores instead

Relates

@picnixz
Copy link
Member

picnixz commented Jan 1, 2024

For this one I prefer sticking to whatever ruff suggests directly. I don't think we need to be overly verbose (but maybe they do suggest it, I haven't checked actually)

@DimitriPapadopoulos
Copy link
Contributor Author

Yes, the whole ruff documentation has been updated.

I cannot give a specific page, all relevant pages refer to the new lint and format sections:

@picnixz
Copy link
Member

picnixz commented Jan 1, 2024

Then I don't mind the change! @AA-Turner do you agree with me? or do you have any reason to keep old syntax until it's no longer supported?

@AA-Turner AA-Turner merged commit 002098b into sphinx-doc:master Jan 1, 2024
@DimitriPapadopoulos DimitriPapadopoulos deleted the lint branch January 2, 2024 08:24
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 2, 2024
@AA-Turner AA-Turner added this to the 7.3.0 milestone Jul 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants