Skip to content
This repository has been archived by the owner on Feb 11, 2025. It is now read-only.

Add F5, F6, F8 and F9 rules to ruff #196

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Conversation

johannaengland
Copy link
Contributor

@johannaengland johannaengland commented Oct 22, 2024

They are already fulfilled on the current codebase.

Tagging as discussion to invite discussion if/which of these rules we want.

Reference: https://docs.astral.sh/ruff/rules/#pyflakes-f

@johannaengland johannaengland self-assigned this Oct 22, 2024
@johannaengland johannaengland changed the title Add F6 rules to ruff Add F6, F8 and F9 rules to ruff Oct 22, 2024
Copy link
Member

@lunkwill42 lunkwill42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm all for it :)

@johannaengland johannaengland changed the title Add F6, F8 and F9 rules to ruff Add F5, F6, F8 and F9 rules to ruff Oct 28, 2024
@johannaengland
Copy link
Contributor Author

Merging since this was approved in a live code review.

@johannaengland johannaengland merged commit 82ba3d9 into main Oct 28, 2024
3 checks passed
@johannaengland johannaengland deleted the linting/add-f6-rules branch October 28, 2024 11:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants