Skip to content

Commit 47479f1

Browse files
committed
Refactor summarize recipe to be easier to read
Most recipes in `justfile` that use `cargo nextest run` write it out in a command, even though a `nextest` recipe exists. This is often done because the command appears in a non-leading line of the recipe, so having the recipe depend on the `nextest` recipe would not be feasible. However, it also has the benefit of being more clear about exactly what command is being run, especially when the arguments are long and complicated, as in `summarize`. This changes `summarize` so that it invokes nextest explicitly as a step of the recipe, rather than depending on `nextest` and passing its arguments through.
1 parent 8bec4a2 commit 47479f1

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

justfile

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -243,7 +243,9 @@ audit:
243243
nextest *FLAGS='--workspace':
244244
cargo nextest run {{ FLAGS }}
245245

246-
summarize EXPRESSION='all()': (nextest '--workspace --run-ignored all --no-fail-fast --status-level none --final-status-level none -E' quote(EXPRESSION))
246+
summarize EXPRESSION='all()':
247+
cargo nextest run --workspace --run-ignored all --no-fail-fast \
248+
--status-level none --final-status-level none -E {{ quote(EXPRESSION) }}
247249

248250
# run nightly rustfmt for its extra features, but check that it won't upset stable rustfmt
249251
fmt:

0 commit comments

Comments
 (0)