Skip to content

Commit 8bec4a2

Browse files
committed
Slightly improve quoting style clarity in justfile
This makes two small changes to the style in which quoting is expressed in the `justfile`: - Outside of commands run by a shell (i.e. when `just` interprets the quotes), prefer single quotes over double quotes when there is both no intent for `\`-escape interpretation to occur and no other reason to use `"` (such as a literal `'`). - Inside commands run by a shel (i.e. when the quoting is shell syntax rather than `just` syntax), omit double quotes around a few literal arguments that contain no `$` nor other characters the shell treats specially. They are alredy omitted in most commands similar to these.
1 parent 4913663 commit 8bec4a2

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

justfile

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ check:
144144
cargo check --no-default-features --features max-control
145145

146146
# Run cargo doc on all crates
147-
doc $RUSTDOCFLAGS="-D warnings":
147+
doc $RUSTDOCFLAGS='-D warnings':
148148
cargo doc --workspace --no-deps --features need-more-recent-msrv
149149
cargo doc --features=max,lean,small --workspace --no-deps --features need-more-recent-msrv
150150

@@ -158,9 +158,9 @@ unit-tests:
158158
cargo nextest run -p gix-archive --features tar
159159
cargo nextest run -p gix-archive --features tar_gz
160160
cargo nextest run -p gix-archive --features zip
161-
cargo nextest run -p gix-status-tests --features "gix-features-parallel"
162-
cargo nextest run -p gix-worktree-state-tests --features "gix-features-parallel"
163-
cargo nextest run -p gix-worktree-tests --features "gix-features-parallel"
161+
cargo nextest run -p gix-status-tests --features gix-features-parallel
162+
cargo nextest run -p gix-worktree-state-tests --features gix-features-parallel
163+
cargo nextest run -p gix-worktree-tests --features gix-features-parallel
164164
cd gix-object; \
165165
set -ex; \
166166
cargo nextest run; \
@@ -172,10 +172,10 @@ unit-tests:
172172
cargo nextest run -p gix-odb-tests --features gix-features-parallel
173173
cargo nextest run -p gix-pack --all-features
174174
cargo nextest run -p gix-pack-tests --features all-features
175-
cargo nextest run -p gix-pack-tests --features "gix-features-parallel"
176-
cargo nextest run -p gix-index-tests --features "gix-features-parallel"
175+
cargo nextest run -p gix-pack-tests --features gix-features-parallel
176+
cargo nextest run -p gix-index-tests --features gix-features-parallel
177177
cargo nextest run -p gix-packetline --features blocking-io,maybe-async/is_sync --test blocking-packetline
178-
cargo nextest run -p gix-packetline --features "async-io" --test async-packetline
178+
cargo nextest run -p gix-packetline --features async-io --test async-packetline
179179
cargo nextest run -p gix-transport --features http-client-curl,maybe-async/is_sync
180180
cargo nextest run -p gix-transport --features http-client-reqwest,maybe-async/is_sync
181181
cargo nextest run -p gix-transport --features async-client
@@ -192,10 +192,10 @@ unit-tests-flaky:
192192
cargo test -p gix --features async-network-client-async-std
193193

194194
target_dir := `cargo metadata --format-version 1 | jq -r .target_directory`
195-
ein := quote(target_dir / "debug/ein")
196-
gix := quote(target_dir / "debug/gix")
197-
jtt := quote(target_dir / "debug/jtt")
198-
it := quote(target_dir / "debug/it")
195+
ein := quote(target_dir / 'debug/ein')
196+
gix := quote(target_dir / 'debug/gix')
197+
jtt := quote(target_dir / 'debug/jtt')
198+
it := quote(target_dir / 'debug/it')
199199

200200
# run journey tests (max)
201201
journey-tests:
@@ -240,10 +240,10 @@ audit:
240240
cargo deny check advisories bans licenses sources
241241

242242
# run tests with `cargo nextest` (all unit-tests, no doc-tests, faster)
243-
nextest *FLAGS="--workspace":
243+
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()': (nextest '--workspace --run-ignored all --no-fail-fast --status-level none --final-status-level none -E' quote(EXPRESSION))
247247

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

0 commit comments

Comments
 (0)