Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Disable max number of arguments Ruff Linting Rule [PLR0913] #155

Open
AntonioMrtz opened this issue Jul 3, 2024 · 1 comment
Open

Disable max number of arguments Ruff Linting Rule [PLR0913] #155

AntonioMrtz opened this issue Jul 3, 2024 · 1 comment
Labels
backend Backend related issues chore Minor changes on config , dependencies etc. No production code modified. good first issue Good for newcomers hacktoberfest hacktoberfest help-wanted Help is wanted python Pull requests that update Python code

Comments

@AntonioMrtz
Copy link
Owner

AntonioMrtz commented Jul 3, 2024

Description

Disable backend linting rule that affects max number of arguments in backend functions. More info on ruff PLR0913 rule

Context

Rule is too strict and doesnt apply correctly to our use case. This forces us to disable the rule in some functions.

How to do it

  • Exclude rule PLR0913 in ruff.toml in section:

ignore = ["PLR0402","PLR0911"]

  • Delete PLR0913 no qa comments in the code base

Testing

Run Backend tests to check if everything is working correctly

@AntonioMrtz AntonioMrtz added good first issue Good for newcomers backend Backend related issues chore Minor changes on config , dependencies etc. No production code modified. help-wanted Help is wanted python Pull requests that update Python code labels Jul 3, 2024
@AntonioMrtz
Copy link
Owner Author

@AntonioMrtz AntonioMrtz pinned this issue Jul 7, 2024
@AntonioMrtz AntonioMrtz unpinned this issue Aug 14, 2024
@AntonioMrtz AntonioMrtz pinned this issue Sep 30, 2024
@AntonioMrtz AntonioMrtz added the hacktoberfest hacktoberfest label Oct 16, 2024
@AntonioMrtz AntonioMrtz unpinned this issue Oct 19, 2024
@AntonioMrtz AntonioMrtz pinned this issue Nov 21, 2024
@AntonioMrtz AntonioMrtz unpinned this issue Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend related issues chore Minor changes on config , dependencies etc. No production code modified. good first issue Good for newcomers hacktoberfest hacktoberfest help-wanted Help is wanted python Pull requests that update Python code
Projects
None yet
Development

No branches or pull requests

2 participants