diff --git a/crates/turborepo-boundaries/src/imports.rs b/crates/turborepo-boundaries/src/imports.rs index 4f88257121013..63870e91a7a1e 100644 --- a/crates/turborepo-boundaries/src/imports.rs +++ b/crates/turborepo-boundaries/src/imports.rs @@ -111,7 +111,9 @@ fn check_import_as_tsconfig_path_alias( return Ok((false, None)); } - let dir = file_path.parent().expect("file_path must have a parent"); + let dir = file_path + .parent() + .ok_or_else(|| Error::NoParentDir(file_path.to_owned()))?; match resolver.resolve(dir, import) { Ok(resolution) => { @@ -395,7 +397,7 @@ pub(crate) fn check_package_import( }); } let package_node = PackageNode::Workspace(PackageName::Other(package_name.to_string())); - let folder = file_path.parent().expect("file_path should have a parent"); + let folder = file_path.parent()?; let is_valid_dependency = dependency_locations.is_dependency(&package_node); if !is_valid_dependency diff --git a/crates/turborepo-boundaries/src/lib.rs b/crates/turborepo-boundaries/src/lib.rs index 0cf08453dbdb8..34290056f148d 100644 --- a/crates/turborepo-boundaries/src/lib.rs +++ b/crates/turborepo-boundaries/src/lib.rs @@ -1,5 +1,5 @@ #![allow(clippy::sliced_string_as_bytes)] -#![allow(clippy::expect_used, clippy::unwrap_used)] +#![allow(clippy::unwrap_used)] // miette's derive macro causes false positives for these lints #![allow(unused_assignments)]