mirror of
https://github.com/pnpm/pnpm.git
synced 2026-07-02 11:55:17 -04:00
fix(pacquet/config): satisfy dylint perfectionist lints (#11672)
Two pre-existing perfectionist findings landed via #11526 and broke the Dylint CI job on main: - `perfectionist::macro-trailing-comma` flagged a trailing comma in a single-line `assert_eq!(...)` that `cargo fmt` had collapsed from a multi-line form without dropping the comma. Removed the comma. - `perfectionist::unicode_ellipsis_in_comments` warned about a U+2026 `…` character in a test comment. CI runs with `RUSTFLAGS=-D warnings`, so the warning fails the build. Replaced with ASCII `...`. Verified locally with `cargo dylint --all -- --all-targets --workspace` under `RUSTFLAGS=-D warnings`; passes clean. --- Written by an agent (Claude Code, claude-opus-4-7).
This commit is contained in:
@@ -319,6 +319,6 @@ mod tests {
|
||||
fn collects_every_unresolved_placeholder_occurrence() {
|
||||
let (value, unresolved) = env_replace_lossy::<NoEnv>("${A}-${B}-${A}");
|
||||
assert_eq!(value, "--");
|
||||
assert_eq!(unresolved, vec!["${A}".to_owned(), "${B}".to_owned(), "${A}".to_owned()],);
|
||||
assert_eq!(unresolved, vec!["${A}".to_owned(), "${B}".to_owned(), "${A}".to_owned()]);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -143,7 +143,7 @@ fn env_replace_substitutes_token() {
|
||||
#[test]
|
||||
fn env_replace_failure_warns_and_drops_unresolved_to_empty() {
|
||||
// Mirrors pnpm's `substituteEnv` lossy fallback: unresolved `${VAR}` becomes
|
||||
// "" so a downstream `Authorization: Bearer …` header is never sent with a
|
||||
// "" so a downstream `Authorization: Bearer ...` header is never sent with a
|
||||
// literal placeholder. See https://github.com/pnpm/pnpm/issues/11513.
|
||||
let ini = "//reg.com/:_authToken=${MISSING}\n";
|
||||
let auth = NpmrcAuth::from_ini::<NoEnv>(ini);
|
||||
|
||||
Reference in New Issue
Block a user