refactor: rename helper function

src/lib.rs

@@ -101,7 +101,8 @@
 //!
 //! The first method [make_decision](crate::solver::DependencyProvider::make_decision) chooses a
 //! package and available version compatible with the provided options.
-//! A helper function [make_fewest_versions_decision_helper](crate::solver::make_fewest_versions_decision_helper)
+//! A helper function
+//! [choose_package_with_fewest_versions](crate::solver::choose_package_with_fewest_versions)
 //! is provided for convenience
 //! in cases when lists of available versions for packages are easily obtained.
 //! The strategy of that helper function consists in choosing the package

src/solver.rs

@@ -234,7 +234,7 @@ pub trait DependencyProvider<P: Package, V: Version> {
     /// > since these packages will run out of versions to try more quickly.
     /// > But there's likely room for improvement in these heuristics.
     ///
-    /// A helper function [make_fewest_versions_decision_helper] is provided to ease
+    /// A helper function [choose_package_with_fewest_versions] is provided to ease
     /// implementations of this method if you can produce an iterator
     /// of the available versions in preference order for any package.
     ///
@@ -268,7 +268,7 @@ pub trait DependencyProvider<P: Package, V: Version> {
 /// The helper finds the package from the `packages` argument with the fewest versions from
 /// `list_available_versions` contained in the constraints. Then takes that package and finds the
 /// first version contained in the constraints.
-pub fn make_fewest_versions_decision_helper<P: Package, V: Version, T, U, I, F>(
+pub fn choose_package_with_fewest_versions<P: Package, V: Version, T, U, I, F>(
     list_available_versions: F,
     potential_packages: impl Iterator<Item = (T, U)>,
 ) -> (T, Option<V>)
@@ -360,7 +360,7 @@ impl<P: Package, V: Version> DependencyProvider<P, V> for OfflineDependencyProvi
         &self,
         potential_packages: impl Iterator<Item = (T, U)>,
     ) -> Result<(T, Option<V>), Box<dyn Error>> {
-        Ok(make_fewest_versions_decision_helper(
+        Ok(choose_package_with_fewest_versions(
             |p| {
                 self.dependencies
                     .get(p)

tests/proptest.rs

@@ -7,7 +7,7 @@ use pubgrub::package::Package;
 use pubgrub::range::Range;
 use pubgrub::report::{DefaultStringReporter, Reporter};
 use pubgrub::solver::{
-    make_fewest_versions_decision_helper, resolve, Dependencies, DependencyProvider,
+    choose_package_with_fewest_versions, resolve, Dependencies, DependencyProvider,
     OfflineDependencyProvider,
 };
 use pubgrub::version::{NumberVersion, Version};
@@ -31,7 +31,7 @@ impl<P: Package, V: Version> DependencyProvider<P, V> for OldestVersionsDependen
         &self,
         potential_packages: impl Iterator<Item = (T, U)>,
     ) -> Result<(T, Option<V>), Box<dyn Error>> {
-        Ok(make_fewest_versions_decision_helper(
+        Ok(choose_package_with_fewest_versions(
             |p| self.0.versions(p).into_iter().flatten().cloned(),
             potential_packages,
         ))