misc: remove funcs and add some docstrings

crates/pixi_git/src/lib.rs

@@ -73,22 +73,6 @@ impl GitUrl {
     pub fn precise(&self) -> Option<GitSha> {
         self.precise
     }
-
-    /// Determines if the given URL looks like a Git URL.
-    pub fn is_git_url(url: &Url) -> bool {
-        // Check if the scheme indicates it's a Git URL.
-        if url.scheme().starts_with("git+") {
-            // Check if the path ends with .git or has a Git-specific format.
-            return true;
-        } else if let Some(path) = url.path_segments() {
-            if path.clone().any(|segment| segment.ends_with(".git")) {
-                return true;
-            }
-        };
-
-        // If the URL doesn't match Git-specific patterns, return false.
-        false
-    }
 }
 
 impl TryFrom<Url> for GitUrl {

crates/pixi_spec/src/lib.rs

@@ -20,7 +20,6 @@ pub use detailed::DetailedSpec;
 pub use git::{GitSpec, Reference};
 use itertools::Either;
 pub use path::{PathBinarySpec, PathSourceSpec, PathSpec};
-use pixi_git::GitUrl;
 use rattler_conda_types::{
     ChannelConfig, NamedChannelOrUrl, NamelessMatchSpec, ParseChannelError, VersionSpec,
 };
@@ -103,21 +102,11 @@ impl PixiSpec {
         channel_config: &ChannelConfig,
     ) -> Self {
         if let Some(url) = spec.url {
-            // check if it is a git url
-            if GitUrl::is_git_url(&url) {
-                let git_url = GitUrl::try_from(url.clone()).unwrap();
-                Self::Git(GitSpec {
-                    git: git_url.repository().clone(),
-                    rev: Some(git_url.reference().clone().into()),
-                    subdirectory: spec.subdir,
-                })
-            } else {
-                Self::Url(UrlSpec {
-                    url,
-                    md5: spec.md5,
-                    sha256: spec.sha256,
-                })
-            }
+            Self::Url(UrlSpec {
+                url,
+                md5: spec.md5,
+                sha256: spec.sha256,
+            })
         } else if spec.build.is_none()
             && spec.build_number.is_none()
             && spec.file_name.is_none()

src/cli/add.rs

@@ -102,8 +102,8 @@ pub async fn execute(args: Args) -> miette::Result<()> {
 
     let (match_specs, source_specs, pypi_deps) = match dependency_config.dependency_type() {
         DependencyType::CondaDependency(spec_type) => {
-            // now if user passed some git configuration
-            // we will use it to create pixi specs
+            // if user passed some git configuration
+            // we will use it to create pixi source specs
             let passed_specs: IndexMap<PackageName, (MatchSpec, SpecType)> = dependency_config
                 .specs()?
                 .into_iter()

src/cli/cli_config.rs

@@ -148,20 +148,24 @@ pub struct GitRev {
 }
 
 impl GitRev {
+    /// Create a new `GitRev`
     pub fn new() -> Self {
         Default::default()
     }
 
+    /// Set the branch
     pub fn with_branch(mut self, branch: String) -> GitRev {
         self.branch = Some(branch);
         self
     }
 
+    /// Set the revision
     pub fn with_rev(mut self, rev: String) -> GitRev {
         self.rev = Some(rev);
         self
     }
 
+    /// Set the tag
     pub fn with_tag(mut self, tag: String) -> GitRev {
         self.tag = Some(tag);
         self