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 /subcmds/sync.py | |
| parent | 3f3c681a02ed27663e06c35328b494b9245881b5 (diff) | |
| download | git-repo-573983948ae0550b0f655ea81431bb08ce0bd540.tar.gz git-repo-573983948ae0550b0f655ea81431bb08ce0bd540.zip | |
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 'subcmds/sync.py')
| -rw-r--r-- | subcmds/sync.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/subcmds/sync.py b/subcmds/sync.py index 03c0eba64..e5fb3bcdb 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py @@ -947,7 +947,7 @@ later is required to fix a server side protocol bug. "sync_dict" ] = multiprocessing.Manager().dict() - objdir_project_map = dict() + objdir_project_map = {} for index, project in enumerate(projects): objdir_project_map.setdefault(project.objdir, []).append(index) projects_list = list(objdir_project_map.values()) @@ -2657,7 +2657,7 @@ later is required to fix a server side protocol bug. if previously_pending_relpaths == pending_relpaths: stalled_projects_str = "\n".join( f" - {path}" - for path in sorted(list(pending_relpaths)) + for path in sorted(pending_relpaths) ) logger.error( "The following projects failed and could " |
