PEP8 enabled

[skupriienko]

Links

• DE-1038

Plan to enable Python conventions

PEP-8

• Use double " quotes
• Use autopep8 and flake8 (with some additional plugins) as initial PEP8 linters
• Use ruff for linting and formatting (add pyproject.toml with the ruff configs)
• Sort imports by ruff check --select I --fix
• Add bandit configs to pyproject.toml (no issues found by running bandit -c pyproject.toml -r .)
• Fix the exception's usage
• Apply linting rules:
  • [x] EXE001 Shebang is present but file is not executable
  • [x] UP009 [*] UTF-8 encoding declaration is unnecessary
  • [x] RET503 Missing explicit return at the end of function able to return non-None value
  • [x] UP004 [*] Class Config inherits from object
  • [x] SIM910 [*] Use kwargs.get("version") instead of kwargs.get("version", None)
  • [ ] SKIP IT: RET504 Unnecessary assignment to response before return statement
  • [x] B904 Within an except clause, raise exceptions with raise ... from err or raise ... from None to distinguish them from errors in exception handling
  • [x] UP032 [*] Use f-string instead of format call
  • [x] RUF010 [*] Use explicit conversion flag
  • [x] G002 Logging statement uses %
  • [x] S101 Use of assert detected
  • [x] UP030 Use implicit references for positional format fields
  • [x] Q003 [*] Change outer quotes to avoid escaping inner quotes
  • [x] Use Path(filename).read_text() if you read the text file to a variable
  • [x] PTH100 os.path.abspath() should be replaced by Path.resolve()
  • [x] PTH120 os.path.dirname() should be replaced by Path.parent
  • [ ] Skip temporarily TRY300 Consider moving response to an 'else' block; use secrets instead of random; add new linting and skipping rules
  • [x] UP015 [*] Unnecessary open mode parameters
  • [x] UP034 [*] Avoid extraneous parentheses
  • [x] S311 Standard pseudo-random generators are not suitable for cryptographic purposes: use secrets.choice()
  • [ ] Skip temporarily ARG001
  • [x] F401 imported but unused
  • [x] Fix E122, E125, E126, E131, E202, E501 (88 symbols)
  • [ ] Skip the linting rule PLR0917 Too many positional arguments
  • [x] Fix E1101: Instance of 'Endpoint' has no '_doc' member (no-member)
  • [x] Fix C0325: Unnecessary parens after 'if' keyword

[skupriienko]

@DanyilNefodov Could you please run tests against these changes? Because I'm getting KeyError: 'MJ_APIKEY_PUBLIC' if I run pytest test.py

[skupriienko]

@DanyilNefodov Could you please run tests against these changes? Because I'm getting KeyError: 'MJ_APIKEY_PUBLIC' if I run pytest test.py

I was able to run tests

[skupriienko]

I reverted Client error handling to see the whole traceback. Also, I reverted random import. I think for tests, it's fine to use random instead of secrets. And I solved the merge conflict. @DanyilNefodov Could you please look at the PR?