Remove unused code from changeset creation

Signed-off-by: Azeem Shaikh <azeemshaikh38@gmail.com>
ossf · Mar 21, 2023 · a7bc2e0 · a7bc2e0
1 parent dfc2439
commit a7bc2e0
Show file tree

Hide file tree

Showing 2 changed files with 214 additions and 243 deletions.
diff --git a/checks/raw/code_review.go b/checks/raw/code_review.go
@@ -183,19 +183,7 @@ func getChangesets(commits []clients.Commit) []checker.Changeset {
 
 	// Changesets are returned in map order (i.e. randomized)
 	for ri := range changesetsByRevInfo {
-		// Ungroup all commits that don't have revision info
-		cs := changesetsByRevInfo[ri]
-		missing := revisionInfo{}
-		if ri == missing {
-			for i := range cs.Commits {
-				c := cs.Commits[i]
-				changesets = append(changesets, checker.Changeset{
-					Commits: []clients.Commit{c},
-				})
-			}
-		} else {
-			changesets = append(changesets, cs)
-		}
+		changesets = append(changesets, changesetsByRevInfo[ri])
 	}
 
 	return changesets