Skip to content

Commit 783dd14

Browse files
[test optimization] Cache for codeowners (#6847)
1 parent 4c487c4 commit 783dd14

File tree

1 file changed

+10
-1
lines changed
  • packages/dd-trace/src/plugins/util

1 file changed

+10
-1
lines changed

packages/dd-trace/src/plugins/util/test.js

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -627,20 +627,29 @@ function getCodeOwnersFileEntries (rootDir) {
627627
return entries.reverse()
628628
}
629629

630+
const codeOwnersPerFileName = new Map()
631+
630632
function getCodeOwnersForFilename (filename, entries) {
631633
if (!entries) {
632634
return null
633635
}
636+
if (codeOwnersPerFileName.has(filename)) {
637+
return codeOwnersPerFileName.get(filename)
638+
}
634639
for (const entry of entries) {
635640
try {
636641
const isResponsible = ignore().add(entry.pattern).ignores(filename)
637642
if (isResponsible) {
638-
return JSON.stringify(entry.owners)
643+
const codeOwners = JSON.stringify(entry.owners)
644+
codeOwnersPerFileName.set(filename, codeOwners)
645+
return codeOwners
639646
}
640647
} catch {
648+
codeOwnersPerFileName.set(filename, null)
641649
return null
642650
}
643651
}
652+
codeOwnersPerFileName.set(filename, null)
644653
return null
645654
}
646655

0 commit comments

Comments
 (0)