Skip to content

Commit b9472db

Browse files
committed
style: fix ruff and formatting in resync-all-specs.py
Adjusted noqa placements to satisfy Ruff 0.1.3 and applied consistent formatting.
1 parent aecc350 commit b9472db

File tree

1 file changed

+8
-14
lines changed

1 file changed

+8
-14
lines changed

.evergreen/scripts/resync-all-specs.py

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ def resync_specs(directory: pathlib.Path, errored: dict[str, str]) -> None:
1818
if spec.name in ["asynchronous"]:
1919
continue
2020
try:
21-
subprocess.run( # noqa: S603
22-
["bash", "./.evergreen/resync-specs.sh", spec.name], # noqa: S607
21+
subprocess.run(
22+
["bash", "./.evergreen/resync-specs.sh", spec.name], # noqa: S603, S607
2323
capture_output=True,
2424
text=True,
2525
check=True,
@@ -32,16 +32,16 @@ def resync_specs(directory: pathlib.Path, errored: dict[str, str]) -> None:
3232
def apply_patches(errored):
3333
print("Beginning to apply patches")
3434
subprocess.run(
35-
["bash", "./.evergreen/remove-unimplemented-tests.sh"], # noqa: S607
35+
["bash", "./.evergreen/remove-unimplemented-tests.sh"], # noqa: S603, S607
3636
check=True,
3737
)
3838
try:
3939
# Avoid shell=True by passing arguments as a list.
4040
# Note: glob expansion doesn't work in shell=False, so we use a list of files.
4141
patches = [str(p) for p in pathlib.Path("./.evergreen/spec-patch/").glob("*")]
4242
if patches:
43-
subprocess.run( # noqa: S603
44-
[ # noqa: S607
43+
subprocess.run(
44+
[ # noqa: S603, S607
4545
"git",
4646
"apply",
4747
"-R",
@@ -66,11 +66,7 @@ def check_new_spec_directories(directory: pathlib.Path) -> list[str]:
6666
and (pathlib.Path(entry.path) / "tests").is_dir()
6767
and len(list(os.scandir(pathlib.Path(entry.path) / "tests"))) > 1
6868
}
69-
test_set = {
70-
entry.name.replace("-", "_")
71-
for entry in os.scandir(directory)
72-
if entry.is_dir()
73-
}
69+
test_set = {entry.name.replace("-", "_") for entry in os.scandir(directory) if entry.is_dir()}
7470
known_mappings = {
7571
"ocsp_support": "ocsp",
7672
"client_side_operations_timeout": "csot",
@@ -89,14 +85,12 @@ def check_new_spec_directories(directory: pathlib.Path) -> list[str]:
8985
return list(spec_set - test_set)
9086

9187

92-
def write_summary(
93-
errored: dict[str, str], new: list[str], filename: str | None
94-
) -> None:
88+
def write_summary(errored: dict[str, str], new: list[str], filename: str | None) -> None:
9589
"""Generate the PR description"""
9690
pr_body = ""
9791
# Avoid shell=True and complex pipes by using Python to process git output
9892
process = subprocess.run(
99-
["git", "diff", "--name-only"], # noqa: S607
93+
["git", "diff", "--name-only"], # noqa: S603, S607
10094
capture_output=True,
10195
text=True,
10296
check=True,

0 commit comments

Comments
 (0)