diff options
| author | Carlos Fernandez <carlosfsanz@meta.com> | 2026-03-26 13:46:38 -0700 |
|---|---|---|
| committer | LUCI <gerrit-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2026-04-03 07:50:52 -0700 |
| commit | 573983948ae0550b0f655ea81431bb08ce0bd540 (patch) | |
| tree | 1065ee4ce7373f8bb164b5c66b824e141a74082c /ssh.py | |
| parent | 3f3c681a02ed27663e06c35328b494b9245881b5 (diff) | |
| download | git-repo-573983948ae0550b0f655ea81431bb08ce0bd540.tar.gz | |
Fix all flake8 warnings from newer flake8-bugbear and flake8-comprehensions
Address warnings introduced by flake8-bugbear 24.12.12 and
flake8-comprehensions 3.16.0:
- C408: Replace dict()/list() calls with literal {} and []
- C413: Remove unnecessary list() around sorted()
- C414: Remove unnecessary list() inside sorted()
- C419: Suppress intentional list comprehension in all() (noqa)
- B001: Replace bare except with except Exception
- B006: Replace mutable default arguments with None
- B010: Replace setattr() with direct attribute assignment
- B017: Use RuntimeError instead of Exception in tests
- B019: Suppress lru_cache on methods for long-lived objects (noqa)
- B033: Remove duplicate item in set literal
Change-Id: If4693d3e946200bbc22f689f7b94da604addcb80
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/566321
Tested-by: Carlos Fernandez <carlosfsanz@meta.com>
Commit-Queue: Carlos Fernandez <carlosfsanz@meta.com>
Reviewed-by: Mike Frysinger <vapier@google.com>
Reviewed-by: Gavin Mak <gavinmak@google.com>
Diffstat (limited to 'ssh.py')
| -rw-r--r-- | ssh.py | 2 |
1 files changed, 1 insertions, 1 deletions
| @@ -149,7 +149,7 @@ class ProxyManager: | |||
| 149 | while True: | 149 | while True: |
| 150 | try: | 150 | try: |
| 151 | procs.pop(0) | 151 | procs.pop(0) |
| 152 | except: # noqa: E722 | 152 | except IndexError: |
| 153 | break | 153 | break |
| 154 | 154 | ||
| 155 | def close(self): | 155 | def close(self): |
