summaryrefslogtreecommitdiff
path: root/subcmds/sync.py
diff options
context:
space:
mode:
authorCarlos Fernandez <carlosfsanz@meta.com>2026-03-26 13:46:38 -0700
committerLUCI <gerrit-scoped@luci-project-accounts.iam.gserviceaccount.com>2026-04-03 07:50:52 -0700
commit573983948ae0550b0f655ea81431bb08ce0bd540 (patch)
tree1065ee4ce7373f8bb164b5c66b824e141a74082c /subcmds/sync.py
parent3f3c681a02ed27663e06c35328b494b9245881b5 (diff)
downloadgit-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.py4
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 "