Restore --reverse flag with deprecation warning

Author: e-q

git-branchless-opts/src/lib.rs

@@ -340,6 +340,12 @@ pub struct SmartlogArgs {
     #[clap(value_parser)]
     pub revset: Option<Revset>,
 
+    /// (Deprecated)
+    /// Print the smartlog in the opposite of the usual order, with the latest
+    /// commits first. Can be configured via `branchless.smartlog.reverse`.
+    #[clap(long)]
+    pub reverse: bool,
+
     /// Don't automatically add HEAD and the main branch to the list of commits
     /// to present. They will still be added if included in the revset.
     #[clap(long)]

git-branchless-smartlog/src/lib.rs

@@ -744,6 +744,11 @@ mod render {
         /// instead.
         pub revset: Option<Revset>,
 
+        /// Deprecated
+        /// Reverse the ordering of items in the smartlog output, list the most
+        /// recent commits first.
+        pub reverse: bool,
+
         /// The options to use when resolving the revset.
         pub resolve_revset_options: ResolveRevsetOptions,
 
@@ -762,6 +767,7 @@ pub fn smartlog(
     let SmartlogOptions {
         event_id,
         revset,
+        reverse,
         resolve_revset_options,
         exact,
     } = options;
@@ -820,9 +826,22 @@ pub fn smartlog(
         exact,
     )?;
 
-    let reverse = get_smartlog_reverse(&repo)?;
+    if reverse {
+        print!(
+            "\
+branchless: WARNING: The `--reverse` flag is deprecated.
+branchless: Please use the `branchless.smartlog.reverse` configuration option.
+"
+        );
+    }
+    let reverse_cfg = get_smartlog_reverse(&repo)?;
+    let reverse_value = match (reverse_cfg, reverse) {
+        (.., true) => true,
+        _ => reverse_cfg,
+    };
+
     let mut lines = render_graph(
-        &effects.reverse_order(reverse),
+        &effects.reverse_order(reverse_value),
         &repo,
         &dag,
         &graph,
@@ -845,7 +864,7 @@ pub fn smartlog(
         ],
     )?
     .into_iter();
-    while let Some(line) = if reverse {
+    while let Some(line) = if reverse_value {
         lines.next_back()
     } else {
         lines.next()
@@ -910,6 +929,7 @@ pub fn command_main(ctx: CommandContext, args: SmartlogArgs) -> EyreExitOr<()> {
         event_id,
         revset,
         resolve_revset_options,
+        reverse,
         exact,
     } = args;
 
@@ -920,6 +940,7 @@ pub fn command_main(ctx: CommandContext, args: SmartlogArgs) -> EyreExitOr<()> {
             event_id,
             revset,
             resolve_revset_options,
+            reverse,
             exact,
         },
     )