Skip to content

Commit 9de03cd

Browse files
committed
fix(add): Catch empty dep name early
1 parent 1791353 commit 9de03cd

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

src/cargo/ops/cargo_add/crate_spec.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
//! Crate name parsing.
22
3+
use anyhow::bail;
34
use anyhow::Context as _;
45

56
use super::Dependency;
@@ -28,6 +29,9 @@ impl CrateSpec {
2829
.map(|(n, v)| (n, Some(v)))
2930
.unwrap_or((pkg_id, None));
3031

32+
if name.is_empty() {
33+
bail!("dependency name cannot be empty");
34+
}
3135
validate_package_name(name, "dependency name", "")?;
3236

3337
if let Some(version) = version {
Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1 @@
1-
thread 'main' panicked at src/cargo/core/dependency.rs:164:9:
2-
assertion failed: !name.is_empty()
3-
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
1+
error: dependency name cannot be empty

0 commit comments

Comments
 (0)