Merge pull request #292 from mergestat/fix-commit-order

patrickdevivo · web-flow · commit 5b6c920656fd · 2022-06-20T18:24:36.000-04:00
fix: use default git log order, instead of `CommitterTime` by default
diff --git a/extensions/internal/git/log.go b/extensions/internal/git/log.go
@@ -208,7 +208,7 @@ func (cur *gitLogCursor) Filter(_ int, s string, values ...sqlite.Value) (err er
 		logger = logger.With().Str("repo-disk-path", path).Logger()
 	}
 
-	var opts = &git.LogOptions{Order: git.LogOrderCommitterTime}
+	var opts = &git.LogOptions{Order: git.LogOrderDefault}
 
 	rev := plumbing.Revision(refName)
 	cur.rev = &rev

Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,7 @@ func (cur *gitLogCursor) Filter(_ int, s string, values ...sqlite.Value) (err er`
`208`	`208`	`logger = logger.With().Str("repo-disk-path", path).Logger()`
`209`	`209`	`}`
`210`	`210`
`211`		`- var opts = &git.LogOptions{Order: git.LogOrderCommitterTime}`
	`211`	`+ var opts = &git.LogOptions{Order: git.LogOrderDefault}`
`212`	`212`
`213`	`213`	`rev := plumbing.Revision(refName)`
`214`	`214`	`cur.rev = &rev`