forever-unstable package only Patch release

Author: JurajSadel

xtask/src/semver_check.rs

@@ -129,9 +129,11 @@ fn required_bump(required_bumps: &[ReleaseType], forever_unstable: bool) -> Rele
         }
     }
 
-    if forever_unstable && min_required_update == ReleaseType::Major {
-        log::warn!("Downgrading required bump from Minor to Patch for unstable package",);
-        min_required_update = ReleaseType::Minor;
+    if forever_unstable
+        && (min_required_update == ReleaseType::Major || min_required_update == ReleaseType::Minor)
+    {
+        log::warn!("Downgrading required bump to Patch for unstable package",);
+        min_required_update = ReleaseType::Patch;
     }
 
     min_required_update
@@ -163,14 +165,27 @@ mod tests {
     }
 
     #[test]
-    fn forever_unstable_downgrades_major_to_minor() {
+    fn forever_unstable_downgrades_major_to_patch() {
         let bumps = [ReleaseType::Major];
 
         let result = required_bump(&bumps, true);
 
         assert_eq!(
             result,
-            ReleaseType::Minor,
+            ReleaseType::Patch,
+            "forever-unstable packages must never require a major bump"
+        );
+    }
+
+    #[test]
+    fn forever_unstable_downgrades_minor_to_patch() {
+        let bumps = [ReleaseType::Minor];
+
+        let result = required_bump(&bumps, true);
+
+        assert_eq!(
+            result,
+            ReleaseType::Patch,
             "forever-unstable packages must never require a major bump"
         );
     }