refactor

- add alias
- support `tags` as alias, invoked without explicit `list`
- always put annotated tag information into `[]`, despite being more noisy.

Author: Byron

gitoxide-core/src/repository/tag.rs

@@ -1,25 +1,27 @@
 pub fn list(repo: gix::Repository, out: &mut dyn std::io::Write) -> anyhow::Result<()> {
     let platform = repo.references()?;
 
-    for mut reference in (platform.tags()?).flatten() {
+    for mut reference in platform.tags()?.flatten() {
         let tag = reference.peel_to_tag();
         let tag_ref = tag.as_ref().map(gix::Tag::decode);
 
-        // `name` is the name of the file in `refs/tags/`. This applies to both lightweight as well
-        // as annotated tags.
+        // `name` is the name of the file in `refs/tags/`.
+        // This applies to both lightweight and annotated tags.
         let name = reference.name().shorten();
-
+        let mut fields = Vec::new();
         match tag_ref {
             Ok(Ok(tag_ref)) => {
-                // `tag_name` is the name provided by the user via `git tag -a/-s/-u`. It is only
-                // present for annotated tags.
-                let tag_name = tag_ref.name;
-
-                if name == tag_name {
-                    writeln!(out, "{name} *")?;
-                } else {
-                    writeln!(out, "{name} [tag name: {}]", tag_ref.name)?;
+                // `tag_name` is the name provided by the user via `git tag -a/-s/-u`.
+                // It is only present for annotated tags.
+                fields.push(format!(
+                    "tag name: {}",
+                    if name == tag_ref.name { "*".into() } else { tag_ref.name }
+                ));
+                if tag_ref.pgp_signature.is_some() {
+                    fields.push("signed".into());
                 }
+
+                writeln!(out, "{name} [{fields}]", fields = fields.join(", "))?;
             }
             _ => {
                 writeln!(out, "{name}")?;

src/plumbing/main.rs

@@ -1304,8 +1304,8 @@ pub fn main() -> Result<()> {
                 },
             ),
         },
-        Subcommands::Tag(cmd) => match cmd {
-            tag::Subcommands::List => prepare_and_run(
+        Subcommands::Tag(platform) => match platform.cmds {
+            Some(tag::Subcommands::List) | None => prepare_and_run(
                 "tag-list",
                 trace,
                 auto_verbose,

src/plumbing/options/mod.rs

@@ -101,8 +101,8 @@ pub enum Subcommands {
     #[clap(subcommand)]
     Commit(commit::Subcommands),
     /// Interact with tag objects.
-    #[clap(subcommand)]
-    Tag(tag::Subcommands),
+    #[clap(visible_alias = "tags")]
+    Tag(tag::Platform),
     /// Verify the integrity of the entire repository
     Verify {
         #[clap(flatten)]
@@ -932,6 +932,12 @@ pub mod commit {
 }
 
 pub mod tag {
+    #[derive(Debug, clap::Parser)]
+    pub struct Platform {
+        #[clap(subcommand)]
+        pub cmds: Option<Subcommands>,
+    }
+
     #[derive(Debug, clap::Subcommand)]
     pub enum Subcommands {
         /// List all tags.