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.

Sign up for GitHub

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

Jump to bottom

Bump ruff to 0.5.0 #467

Merged

Kane610 merged 3 commits into master from bump_ruff_to_0.5.0

Jul 3, 2024

Owner

Kane610 commented Jul 3, 2024

No description provided.

Kane610 added 3 commits

July 3, 2024 15:29


          axis/interfaces/pwdgrp_cgi.py:37:9: SIM103 Return the condition directly

322dac9

   |
35 |       def listed_in_parameters(self) -> bool:
36 |           """Is pwdgrp.cgi supported."""
37 |           if self.vapix.params.property_handler.supported and (
   |  _________^
38 | |             self.vapix.params.property_handler["0"].api_http_version >= 3
39 | |         ):
40 | |             return True
41 | |         return False
   | |____________________^ SIM103
42 |
43 |       async def _api_request(self) -> dict[str, User]:
   |
   = help: Inline condition


          axis/models/ptz_cgi.py:317:17: PERF403 Use a dictionary comprehension…

2a44e0f

… instead of a for-loop

    |
315 |         ):
316 |             if command_enum is not None:
317 |                 data[key] = command_enum
    |                 ^^^^^^^^^^^^^^^^^^^^^^^^ PERF403
318 |
319 |         if self.continuous_pantilt_move:
    |


          Update precommit

01932e5

Kane610 merged commit 2110920 into master

1 check passed

Kane610 deleted the bump_ruff_to_0.5.0 branch

July 3, 2024 13:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet