Skip to content

Commit 9cc2c15

Browse files
committed
fix(auto-merge): Warn if not all PRs could be merged
1 parent d423b30 commit 9cc2c15

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

packages/auto-merge/src/cli.ts

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,9 +70,18 @@ async function runAction(autoMerge: AutoMerge, repositories: Repository[], pullR
7070
return count + repository.actionResults.length;
7171
}, 0);
7272

73-
const prPluralized = pluralize('PR', actedRepositories);
73+
const successRepositories = [...approveResults, ...mergeResults].filter(repository => {
74+
return repository.actionResults.some(result => result.error === undefined);
75+
}).length;
76+
77+
const prPluralized = pluralize('PR', successRepositories);
7478
const doAction = configFileData.autoApprove ? 'Approved and merged' : 'Merged';
75-
logger.info(`${doAction} ${actedRepositories} ${prPluralized} matching "${regex}".`);
79+
const infoMessage = `${doAction} ${successRepositories} ${prPluralized} matching "${regex}".`;
80+
if (actedRepositories === 0) {
81+
logger.warn(infoMessage);
82+
} else {
83+
logger.info(infoMessage);
84+
}
7685
}
7786

7887
function askQuestion(question: string): Promise<string> {

0 commit comments

Comments
 (0)