code_review: PR #1783

It's not necessary to check commit with `null` since the caller will make sure that the target commit exists

Signed-off-by: leo <[email protected]>

Author: love-linger

src/ViewModels/Histories.cs

@@ -242,9 +242,6 @@ public async Task<bool> CheckoutBranchByDecoratorAsync(Models.Decorator decorato
 
         public async Task CheckoutBranchByCommitAsync(Models.Commit commit)
         {
-            if (commit == null)
-                return;
-
             if (commit.IsCurrentHead)
                 return;