apply changes

Author: vshakitskiy

compiler-cli/src/dependencies.rs

@@ -3,7 +3,6 @@ mod dependency_manager;
 use std::{
     cell::RefCell,
     collections::{HashMap, HashSet},
-    io::Write,
     process::Command,
     rc::Rc,
     time::Instant,
@@ -111,7 +110,7 @@ fn get_manifest_details(paths: &ProjectPaths) -> Result<(PackageConfig, Manifest
     Ok((config, manifest))
 }
 
-fn list_manifest_packages<W: Write>(mut buffer: W, manifest: Manifest) -> Result<()> {
+fn list_manifest_packages<W: std::io::Write>(mut buffer: W, manifest: Manifest) -> Result<()> {
     manifest
         .packages
         .into_iter()
@@ -122,7 +121,7 @@ fn list_manifest_packages<W: Write>(mut buffer: W, manifest: Manifest) -> Result
         })
 }
 
-fn list_package_and_dependencies_tree<W: Write>(
+fn list_package_and_dependencies_tree<W: std::io::Write>(
     mut buffer: W,
     options: TreeOptions,
     packages: Vec<ManifestPackage>,

compiler-cli/src/dependencies/snapshots/gleam_cli__dependencies__tests__pretty_print_version_updates.snap

@@ -0,0 +1,8 @@
+---
+source: compiler-cli/src/dependencies/tests.rs
+expression: output
+---
+Package                Current   Latest
+gleam_stdlib           v0.45.0   v0.46.0
+very_long_package_name v12.12.12 v120.12.12
+wisp                   v2.1.0    v2.1.1

compiler-cli/src/dependencies/tests.rs

@@ -1348,3 +1348,27 @@ fn test_pretty_print_major_versions_available() {
 
     insta::assert_snapshot!(output);
 }
+
+#[test]
+fn test_pretty_print_version_updates() {
+    let versions = vec![
+        (
+            "gleam_stdlib".to_string(),
+            (Version::new(0, 45, 0), Version::new(0, 46, 0)),
+        ),
+        (
+            "wisp".to_string(),
+            (Version::new(2, 1, 0), Version::new(2, 1, 1)),
+        ),
+        (
+            "very_long_package_name".to_string(),
+            (Version::new(12, 12, 12), Version::new(120, 12, 12)),
+        ),
+    ]
+    .into_iter()
+    .collect();
+
+    let output = pretty_print_version_updates(versions);
+
+    insta::assert_snapshot!(output);
+}

compiler-core/src/dependency.rs

@@ -84,6 +84,9 @@ fn resolve_versions_diffs(
                 .filter(|version| !version.is_pre())
                 .max()?;
 
+            // If we're checking for major version updates, only include the
+            // package if a new major version is available. Otherwise, include
+            // the package if there is any new version available.
             match check_major_versions {
                 true => {
                     if latest.major <= version.major {