Remove rustfmt --recursive option

stepancheg · facebook-github-bot · commit 0075f6f51d37 · 2022-03-30T16:56:18.000-07:00
Summary: D35234535 updated rustfmt, which no longer has `--recursive` option. `rustfmt` [seems to](rust-lang/rustfmt#5018 (comment)) flip `--recursive` option with `--skip-children`. Reviewed By: bobyangyf Differential Revision: D35240294 fbshipit-source-id: bbcd4dc3556c0753c07f3cbe380553e6426d1527
diff --git a/test.py b/test.py
@@ -118,7 +118,7 @@ def rustfmt(buck2_dir: Path, ci: bool) -> None:
     env["RUSTFMT"] = str(
         buck2_dir.parent.parent / "tools" / "third-party" / "rustfmt" / "rustfmt"
     )
-    output = run([cargo_fmt, "--", "--recursive"], capture_output=True, env=env).stdout
+    output = run([cargo_fmt, "--"], capture_output=True, env=env).stdout
     for line in output.splitlines():
         # The fbsource formatter spews out:
         #   Warning: the `merge_imports` option is deprecated. Use `imports_granularity=Crate` instead

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ def rustfmt(buck2_dir: Path, ci: bool) -> None:`
`118`	`118`	`env["RUSTFMT"] = str(`
`119`	`119`	`buck2_dir.parent.parent / "tools" / "third-party" / "rustfmt" / "rustfmt"`
`120`	`120`	`)`
`121`		`- output = run([cargo_fmt, "--", "--recursive"], capture_output=True, env=env).stdout`
	`121`	`+ output = run([cargo_fmt, "--"], capture_output=True, env=env).stdout`
`122`	`122`	`for line in output.splitlines():`
`123`	`123`	`# The fbsource formatter spews out:`
`124`	`124`	# Warning: the `merge_imports` option is deprecated. Use `imports_granularity=Crate` instead