feat(event): add tags filter to Push struct for event handling (#157)

* feat(event): add tags filter to Push struct for event handling

* refactor: simplify string formatting in various display implementations

* feat(workflow): add tag filter to Push event for versioning

* feat(workflow): add tag support for push events in CI configuration

* feat(workflow): add tag support for push events in autofix CI configuration

Author: amitksingh1490

.github/workflows/autofix.yml

@@ -29,6 +29,8 @@ on:
   push:
     branches:
     - main
+    tags:
+    - v*
 jobs:
   lint:
     name: Lint Fix

.github/workflows/ci.yml

@@ -29,6 +29,8 @@ on:
   push:
     branches:
     - main
+    tags:
+    - v*
 jobs:
   build:
     name: Build and Test

crates/gh-workflow-tailcall/src/standard.rs

@@ -239,7 +239,7 @@ impl StandardWorkflow {
 
     fn workflow_event(&self) -> Event {
         Event::default()
-            .push(Push::default().add_branch("main"))
+            .push(Push::default().add_branch("main").add_tag("v*"))
             .pull_request(
                 PullRequest::default()
                     .add_type(PullRequestType::Opened)

crates/gh-workflow/src/cargo.rs

@@ -74,7 +74,7 @@ impl From<Cargo> for Step<Run> {
         let mut command = vec!["cargo".to_string()];
 
         if let Some(toolchain) = value.toolchain {
-            command.push(format!("+{}", toolchain));
+            command.push(format!("+{toolchain}"));
         }
 
         command.push(value.command);

crates/gh-workflow/src/ctx.rs

@@ -202,25 +202,25 @@ impl fmt::Display for Step {
             Step::Root => write!(f, ""),
             Step::Select { name, object } => {
                 if matches!(**object, Step::Root) {
-                    write!(f, "{}", name)
+                    write!(f, "{name}")
                 } else {
-                    write!(f, "{}.{}", object, name)
+                    write!(f, "{object}.{name}")
                 }
             }
             Step::Eq { left, right } => {
-                write!(f, "{} == {}", left, right)
+                write!(f, "{left} == {right}")
             }
             Step::And { left, right } => {
-                write!(f, "{} && {}", left, right)
+                write!(f, "{left} && {right}")
             }
             Step::Or { left, right } => {
-                write!(f, "{} || {}", left, right)
+                write!(f, "{left} || {right}")
             }
             Step::Literal(value) => {
-                write!(f, "'{}'", value)
+                write!(f, "'{value}'")
             }
             Step::Concat { left, right } => {
-                write!(f, "{}{}", left, right)
+                write!(f, "{left}{right}")
             }
         }
     }

crates/gh-workflow/src/event.rs

@@ -643,6 +643,9 @@ pub struct Push {
     /// Filter on specific file paths
     #[serde(default, skip_serializing_if = "Vec::is_empty")]
     pub paths: Vec<String>,
+    /// Filter on specific tags
+    #[serde(default, skip_serializing_if = "Vec::is_empty")]
+    pub tags: Vec<String>,
 }
 
 impl Push {
@@ -657,6 +660,12 @@ impl Push {
         self.paths.push(path.into());
         self
     }
+
+    /// Adds a tag name to filter on
+    pub fn add_tag<S: Into<String>>(mut self, tag: S) -> Self {
+        self.tags.push(tag.into());
+        self
+    }
 }
 
 /// Types of registry package events

crates/gh-workflow/src/generate.rs

@@ -1,7 +1,6 @@
 //! This module provides functionality to customize generation of the GitHub
 //! Actions workflow files.
 
-use std::io::ErrorKind;
 use std::path::PathBuf;
 use std::process::Command;
 
@@ -70,7 +69,7 @@ impl Generate {
                 }
                 Err(Error::MissingWorkflowFile(path)) => {
                     std::fs::create_dir_all(path.parent().ok_or(Error::IO(
-                        std::io::Error::new(ErrorKind::Other, "Invalid parent dir(s) path"),
+                        std::io::Error::other("Invalid parent dir(s) path"),
                     ))?)?;
                     std::fs::write(path.clone(), content)?;
                     println!("Generated workflow file: {}", path.display());
@@ -113,13 +112,13 @@ fn organize_job_dependency(mut workflow: Workflow) -> Workflow {
                     job_ids.push(id.to_owned());
                 } else {
                     // Create a job-id for the job
-                    let id = format!("job-{}", job_id);
+                    let id = format!("job-{job_id}");
 
                     // Add job id as the dependency
                     job_ids.push(id.clone());
 
                     // Insert the missing job into the new_jobs
-                    new_jobs.insert(format!("job-{}", job_id), job.clone());
+                    new_jobs.insert(format!("job-{job_id}"), job.clone());
 
                     job_id += 1;
                 }

crates/gh-workflow/src/rust_flag.rs

@@ -56,14 +56,14 @@ impl Display for RustFlags {
     fn fmt(&self, f: &mut Formatter<'_>) -> std::fmt::Result {
         match self {
             RustFlags::Lint(name, lint) => match lint {
-                Lint::Allow => write!(f, "-A{}", name),
-                Lint::Warn => write!(f, "-W{}", name),
-                Lint::Deny => write!(f, "-D{}", name),
-                Lint::Forbid => write!(f, "-F{}", name),
-                Lint::Codegen => write!(f, "-C{}", name),
-                Lint::Experiment => write!(f, "-Z{}", name),
+                Lint::Allow => write!(f, "-A{name}"),
+                Lint::Warn => write!(f, "-W{name}"),
+                Lint::Deny => write!(f, "-D{name}"),
+                Lint::Forbid => write!(f, "-F{name}"),
+                Lint::Codegen => write!(f, "-C{name}"),
+                Lint::Experiment => write!(f, "-Z{name}"),
             },
-            RustFlags::Combine(lhs, rhs) => write!(f, "{} {}", lhs, rhs),
+            RustFlags::Combine(lhs, rhs) => write!(f, "{lhs} {rhs}"),
         }
     }
 }

crates/gh-workflow/src/toolchain.rs

@@ -44,7 +44,7 @@ impl Display for Component {
             Component::Rustfmt => "rustfmt",
             Component::RustDoc => "rust-doc",
         };
-        write!(f, "{}", val)
+        write!(f, "{val}")
     }
 }
 
@@ -64,7 +64,7 @@ impl Display for Arch {
             Arch::Arm => "arm",
             Arch::Wasm32 => "wasm32",
         };
-        write!(f, "{}", val)
+        write!(f, "{val}")
     }
 }
 
@@ -82,7 +82,7 @@ impl Display for Vendor {
             Vendor::Apple => "apple",
             Vendor::PC => "pc",
         };
-        write!(f, "{}", val)
+        write!(f, "{val}")
     }
 }
 
@@ -102,7 +102,7 @@ impl Display for System {
             System::Linux => "linux",
             System::Darwin => "darwin",
         };
-        write!(f, "{}", val)
+        write!(f, "{val}")
     }
 }
 
@@ -122,7 +122,7 @@ impl Display for Abi {
             Abi::Msvc => "msvc",
             Abi::Musl => "musl",
         };
-        write!(f, "{}", val)
+        write!(f, "{val}")
     }
 }
 
@@ -205,10 +205,10 @@ impl From<Toolchain> for Step<Use> {
                 Version::Stable => "stable".to_string(),
                 Version::Nightly => "nightly".to_string(),
                 Version::Custom((major, minor, patch)) => {
-                    format!("{}.{}.{}", major, minor, patch)
+                    format!("{major}.{minor}.{patch}")
                 }
             })
-            .reduce(|acc, a| format!("{}, {}", acc, a));
+            .reduce(|acc, a| format!("{acc}, {a}"));
 
         let mut input = Input::default();
 
@@ -233,7 +233,7 @@ impl From<Toolchain> for Step<Use> {
                 .components
                 .iter()
                 .map(|c| c.to_string())
-                .reduce(|acc, a| format!("{}, {}", acc, a))
+                .reduce(|acc, a| format!("{acc}, {a}"))
                 .unwrap_or_default();
 
             input = input.add("components", components);
@@ -252,7 +252,7 @@ impl From<Toolchain> for Step<Use> {
             let cache_workspaces = value
                 .cache_workspaces
                 .iter()
-                .fold("".to_string(), |acc, a| format!("{}\n{}", acc, a));
+                .fold("".to_string(), |acc, a| format!("{acc}\n{a}"));
 
             input = input.add("cache-workspaces", cache_workspaces);
         }