Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Build/add boolean trap ruff rule #70

Merged
merged 3 commits into from
Oct 17, 2023
Merged

Conversation

emilio-desousa
Copy link
Contributor

Hello!

I would like to suggest a new rule that allows us to have better boolean management in functions arguments.
See rule definition

Thank you !

@@ -67,6 +67,7 @@ select = [
"UP", # pyupgrade
"PT", # flake8-pytest-style
"RUF", # Ruff-specific rules
"FBT001", # flake8-boolean-trap
Copy link
Collaborator

Choose a reason for hiding this comment

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

You broke the alignment of all the "#"! 😛

@@ -51,7 +51,7 @@ target-version = ['py<%= pythonMajorVersionShortcut %>']
[tool.ruff]
target-version = 'py<%= pythonMajorVersionShortcut %>'

# See all rules at https://beta.ruff.rs/docs/rules/
# See all rules at https://docs.astral.sh/ruff/rules/
Copy link
Collaborator

Choose a reason for hiding this comment

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

Cool!

@ywolff ywolff merged commit 8cf4eed into main Oct 17, 2023
@ywolff ywolff deleted the build/add-boolean-trap-ruff-rule branch October 17, 2023 12:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants