shithub: furgit

Download patch

ref: 26f7f0e33f667a934532e393f40c984172c15889
parent: 5b8941986d4c3f398fc1fc2d1314e80510be346c
author: Runxi Yu <me@runxiyu.org>
date: Sat Mar 7 16:33:31 EST 2026 

refstore/files: Fix lints


--- a/refstore/files/batch_apply.go
+++ b/refstore/files/batch_apply.go
@@ -38,12 +38,14 @@
 			ops:   []txOp{op},
 		}
 
-		if err := tx.validateOp(op); err != nil {
+		err := tx.validateOp(op)
+		if err != nil {
 			results[i].Error = err
+
 			continue
 		}
 
-		err := tx.Commit()
+		err = tx.Commit()
 		if err == nil {
 			continue
 		}
@@ -50,6 +52,7 @@
 
 		if isBatchRejected(err) {
 			results[i].Error = err
+
 			continue
 		}
 
--- a/refstore/files/batch_test.go
+++ b/refstore/files/batch_test.go
@@ -47,11 +47,13 @@
 			t.Fatalf("valid delete failed: %v", results[1].Error)
 		}
 
-		if _, err := store.Resolve("refs/heads/main"); err != nil {
+		_, err = store.Resolve("refs/heads/main")
+		if err != nil {
 			t.Fatalf("Resolve(main): %v", err)
 		}
 
-		if _, err := store.Resolve("refs/heads/topic"); err == nil {
+		_, err = store.Resolve("refs/heads/topic")
+		if err == nil {
 			t.Fatal("refs/heads/topic still exists")
 		}
 	})
@@ -91,7 +93,8 @@
 			t.Fatal("duplicate ref operation did not report an error")
 		}
 
-		if _, err := store.Resolve("refs/heads/main"); err != nil {
+		_, err = store.Resolve("refs/heads/main")
+		if err != nil {
 			t.Fatalf("Resolve(main): %v", err)
 		}
 	})
-- 
⑨