From 5d0220e62b59fa0a78a024be659417376aca57b2 Mon Sep 17 00:00:00 2001 From: Dimitri Mitropoulos Date: Thu, 13 Feb 2025 09:17:36 -0500 Subject: [PATCH] fix errant lockfile reference --- .../src/change_mapper/package.rs | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/crates/turborepo-repository/src/change_mapper/package.rs b/crates/turborepo-repository/src/change_mapper/package.rs index d09094992414b..7c6d57f96265b 100644 --- a/crates/turborepo-repository/src/change_mapper/package.rs +++ b/crates/turborepo-repository/src/change_mapper/package.rs @@ -5,6 +5,7 @@ use wax::{BuildError, Program}; use crate::{ change_mapper::{AllPackageChangeReason, PackageInclusionReason}, package_graph::{PackageGraph, PackageName, WorkspacePackage}, + package_manager::PackageManager, }; pub enum PackageMapping { @@ -113,10 +114,10 @@ impl PackageChangeMapper for GlobalDepsPackageChangeMapper<'_> { // that we can make this more accurate by checking which package // manager, since not all package managers may permit root pulling from // workspace package dependencies - if matches!( - path.as_str(), - "package.json" | "pnpm-lock.yaml" | "yarn.lock" - ) { + if PackageManager::supported_managers() + .iter() + .any(|pm| pm.lockfile_name() == path.as_str()) + { return PackageMapping::Package(( WorkspacePackage { name: PackageName::Root, @@ -161,10 +162,10 @@ mod tests { change_mapper::{ AllPackageChangeReason, ChangeMapper, PackageChanges, PackageInclusionReason, }, - discovery, - discovery::PackageDiscovery, + discovery::{self, PackageDiscovery}, package_graph::{PackageGraphBuilder, WorkspacePackage}, package_json::PackageJson, + package_manager::PackageManager, }; #[allow(dead_code)] @@ -175,7 +176,7 @@ mod tests { &self, ) -> Result { Ok(discovery::DiscoveryResponse { - package_manager: crate::package_manager::PackageManager::Npm, + package_manager: PackageManager::Npm, workspaces: vec![], }) }