We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
During the PR #5343 revision, I found some Python idiomatic enhancements to implement on the Allocation module.
Reorder import statements
wazuh-qa/deployability/modules/allocation/allocation.py
Lines 5 to 9 in 079f1ae
The Python Linter marks that the order of the sentences does not conform to the PEP 8 standards:
Change the bool comparison
Line 67 in 079f1ae
Use the standard Python comparison if not check_connection:
if not check_connection:
Remove unnecessary f-string
Line 69 in 079f1ae
The text was updated successfully, but these errors were encountered:
c-bordon
No branches or pull requests
Description
During the PR #5343 revision, I found some Python idiomatic enhancements to implement on the Allocation module.
Reorder import statements
wazuh-qa/deployability/modules/allocation/allocation.py
Lines 5 to 9 in 079f1ae
The Python Linter marks that the order of the sentences does not conform to the PEP 8 standards:
Change the bool comparison
wazuh-qa/deployability/modules/allocation/allocation.py
Line 67 in 079f1ae
Use the standard Python comparison
if not check_connection:
Remove unnecessary f-string
wazuh-qa/deployability/modules/allocation/allocation.py
Line 69 in 079f1ae
Replace the f-string with a string.
The text was updated successfully, but these errors were encountered: