dashboard / erock/git-pr / Add range-diff to contrib #73 rss

accepted · opened on 2025-08-21T13:40:42Z by jolheiser
Help
checkout latest patchset:
ssh pr.pico.sh print pr-73 | git am -3
checkout any patchset in a patch request:
ssh pr.pico.sh print ps-X | git am -3
add changes to patch request:
git format-patch main --stdout | ssh pr.pico.sh pr add 73
add review to patch request:
git format-patch main --stdout | ssh pr.pico.sh pr add --review 73
accept PR:
ssh pr.pico.sh pr accept 73
close PR:
ssh pr.pico.sh pr close 73

Logs

jolheiser created pr with ps-145 on 2025-08-21T13:40:42Z
erock changed status on 2025-08-21T13:55:28Z {"status":"accepted"}

Patchsets

ps-145 by jolheiser on 2025-08-21T13:40:42Z

Patchset ps-145

Add range-diff to contrib

jolheiser
2025-08-21T12:02:49Z
Back to top

Add range-diff to contrib

Helpful to be able to see a valid range-diff when doing some dev testing.
As an example, this PR was originally intended for something else that turned out to be me testing range-diffs incorrectly. 😅

Signed-off-by: jolheiser <git@jolheiser.com>
contrib/dev/main.go link
+13 -0
 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
diff --git a/contrib/dev/main.go b/contrib/dev/main.go
index ff5c539..eba4a74 100644
--- a/contrib/dev/main.go
+++ b/contrib/dev/main.go
@@ -56,6 +56,14 @@ func main() {
 	if err != nil {
 		panic(err)
 	}
+	rd1, err := fixtures.Fixtures.ReadFile("a_b_reorder.patch")
+	if err != nil {
+		panic(err)
+	}
+	rd2, err := fixtures.Fixtures.ReadFile("a_c_changed_commit.patch")
+	if err != nil {
+		panic(err)
+	}
 
 	// Accepted patch
 	userKey.MustCmd(patch, "pr create test")
@@ -87,6 +95,11 @@ func main() {
 	userKey.MustCmd(nil, "pr edit 6 Closed patch with review")
 	adminKey.MustCmd(otherPatch, "pr add --close 6")
 
+	// Range Diff
+	userKey.MustCmd(rd1, "pr create test")
+	userKey.MustCmd(nil, "pr edit 7 Range Diff")
+	userKey.MustCmd(rd2, "pr add 7")
+
 	fmt.Println("time to do some testing...")
 	ch := make(chan os.Signal, 1)
 	signal.Notify(ch, os.Interrupt, syscall.SIGTERM)