Default dryrun

_examples/actions/write_a_readme/octopush.yml

@@ -1,12 +1,12 @@
 apiVersion: action
 name: write-a-readme
 select:
-  github:
+  # github:
-    repositories:
+  #   repositories:
-      - kjuulh/octopush-test
+  #     - kjuulh/octopush-test
-    push:
+  #   push:
-      pull-request:
+  #     pull-request:
-        name: "write a readme"
+  #       name: "write a readme"
 
   gitea:
     repositories:
@@ -15,12 +15,12 @@ select:
       pull-request:
         name: "write a readme"
 
-  git:
+  # git:
-    repositories:
+  #   repositories:
-      - git@git.front.kjuulh.io:kjuulh/octopush-test.git
+  #     - git@git.front.kjuulh.io:kjuulh/octopush-test.git
-    push:
+  #   push:
-      branch:
+  #     branch:
-        name: "write a readme"
+  #       name: "write a readme"
 action:
   type: go
   entry: "main.go"

crates/octopush_cli/src/commands/execute.rs

@@ -63,6 +63,7 @@ pub fn execute_cmd() -> Command {
                 .long("dry-run")
                 .action(ArgAction::Set)
                 .env("OCTOPUSH_DRY_RUN")
+                .default_value("true")
                 .required(false),
         )
 }
@@ -78,7 +79,14 @@ pub async fn execute_subcommand(args: &ArgMatches) -> eyre::Result<()> {
 
     let github_http_token = args.get_one::<String>("github-api-token");
     let github_username = args.get_one::<String>("github-username");
-    let dryrun = args.get_one::<bool>("dry-run").unwrap_or(&false).clone();
+    let dryrun: bool = args
+        .get_one::<String>("dry-run")
+        .ok_or(eyre::anyhow!("--dry-run is required"))?
+        .parse()?;
+
+    if dryrun {
+        tracing::info!("running in dry-run mode");
+    }
 
     let service_register = ServiceRegister::new(
         LocalGitProviderOptions { http_auth: None },

crates/octopush_core/src/selectors/git_selector.rs

@@ -43,7 +43,7 @@ impl GitSelector {
             self.executor.execute(&path, action_path, action).await?;
 
             if dryrun {
-                return Ok(());
+                continue;
             }
 
             if let Some(push) = &git.push {

crates/octopush_core/src/selectors/gitea_selector.rs

@@ -49,7 +49,7 @@ impl GiteaSelector {
             self.executor.execute(&path, action_path, action).await?;
 
             if dryrun {
-                return Ok(());
+                continue;
             }
 
             if let Some(push) = &git.push {

crates/octopush_core/src/selectors/github_selector.rs

@@ -49,7 +49,7 @@ impl GitHubSelector {
             self.executor.execute(&path, action_path, action).await?;
 
             if dryrun {
-                return Ok(());
+                continue;
             }
 
             if let Some(push) = &git.push {