Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Eli Barzilay <[email protected]>
  • Loading branch information
jablko and elibarzilay authored Nov 27, 2020
1 parent 87a49f7 commit 7de8392
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions src/compute-pr-actions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -309,13 +309,8 @@ export function process(prInfo: PrInfo | BotEnsureRemovedFromProject | BotNoPack
post({ tag: "welcome", status: createWelcomeComment(info) });

// Propagate suggestions into actions
for (const pkg of info.pkgInfo) {
for (const file of pkg.files) {
if (file.suggestion) {
context.suggestions[file.path] = file.suggestion;
}
}
}
info.pkgInfo.forEach(pkg => pkg.files.forEach(file =>
file.suggestion && context.suggestions[file.path] = file.suggestion));

// Ping reviewers when needed
if (!(info.hasChangereqs || info.approvedBy.includes("owner") || info.approvedBy.includes("maintainer"))) {
Expand Down

0 comments on commit 7de8392

Please sign in to comment.