Skip to content

Commit 98bd9fd

Browse files
committed
Address review comments by @nichmor.
1 parent 9be9053 commit 98bd9fd

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

crates/pixi/tests/integration_rust/pypi_tests.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,7 @@ index-url = "{index_url}"
223223
// But not for osx-arm64
224224
assert!(
225225
!lock.contains_pep508_requirement("default", platform2, nccl_req.clone()),
226-
"default environment shouldn't include nccl for osx-64"
226+
"default environment shouldn't include nccl for osx-arm64"
227227
);
228228
}
229229

crates/pixi_cli/src/workspace/export/conda_environment.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ fn format_pip_dependency(name: &PypiPackageName, requirement: &PixiPypiSpec) ->
120120

121121
let marker_str = markers.try_to_string();
122122
if let Some(marker_str) = marker_str {
123-
dependency.push_str(&format!("; {}", marker_str));
123+
dependency.push_str(&format!("; {marker_str}"));
124124
}
125125

126126
dependency

crates/pixi_pypi_spec/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ fn serialize_markertree<S>(value: &MarkerTree, s: S) -> Result<S::Ok, S::Error>
124124
where
125125
S: serde::Serializer,
126126
{
127-
// `.unwrap()` succeeds because we don't serialize when
127+
// `.expect()` succeeds because we don't serialize when
128128
// `value.is_true()`, which is the default.
129129
value.contents().expect("contents were null").serialize(s)
130130
}

0 commit comments

Comments
 (0)