fix: Is there a way to not upgrade a package if it's peer depndencies…

… are not currently met raineorshine#1418

src/lib/upgradePackageDefinitions.ts

@@ -29,25 +29,24 @@ const checkIfInPeerViolation = (
   upgradedPeerDependencies: Index<Index<VersionSpec>>,
 ): CheckIfInPeerViolationResult => {
   const upgradedDependencies = { ...currentDependencies, ...filteredUpgradedDependencies }
-  const violatedDependencies = new Set<string>()
   const filteredUpgradedDependenciesAfterPeers = pickBy(filteredUpgradedDependencies, (spec, dep) => {
     const peerDeps = upgradedPeerDependencies[dep]
     if (!peerDeps) {
       return true
     }
-    const valid = Object.entries(peerDeps).every(
+    return Object.entries(peerDeps).every(
       ([peer, peerSpec]) =>
         upgradedDependencies[peer] === undefined || intersects(upgradedDependencies[peer], peerSpec),
     )
-    if (!valid) {
-      violatedDependencies.add(dep)
-    }
-    return valid
   })
-  const violated = violatedDependencies.size > 0
+  const violated =
+    Object.keys(filteredUpgradedDependencies).length > Object.keys(filteredUpgradedDependenciesAfterPeers).length
   let filteredUpgradedPeerDependencies = upgradedPeerDependencies
   if (violated) {
-    filteredUpgradedPeerDependencies = pickBy(upgradedPeerDependencies, (spec, dep) => !violatedDependencies.has(dep))
+    filteredUpgradedPeerDependencies = pickBy(
+      upgradedPeerDependencies,
+      (spec, dep) => filteredUpgradedDependenciesAfterPeers[dep] || !filteredUpgradedDependencies[dep],
+    )
   }
   return {
     violated,