diff --git a/src/tasks/remind.ts b/src/tasks/remind.ts index 222480bb..4d72eae7 100644 --- a/src/tasks/remind.ts +++ b/src/tasks/remind.ts @@ -120,6 +120,9 @@ export async function remind(api: Api): Promise { if (mergeRequest.assignee.username === username) { return false; } + if (mergeRequest.assignee.username.includes('openstapps')) { + return false; + } if (approval.approved_by.length === 0) { return true; @@ -135,7 +138,7 @@ export async function remind(api: Api): Promise { }); }) .map((username) => `@${username}`) - .join(', '); + .join(' '); // send message to slack await client?.chat.postMessage({ @@ -143,13 +146,14 @@ export async function remind(api: Api): Promise { text: `Merge request '${mergeRequest.title}' needs more approvals! See ${mergeRequest.web_url}!`, }); - // create note in merge request + // assign reviewers await api.createNote( mergeRequest.project_id, Scope.MERGE_REQUESTS, mergeRequest.iid, - `${NOTE_PREFIX} Please review, ${possibleApprovers}!`, + `/assign_reviewer ${possibleApprovers}`, ); + } else { Logger.log(`Merge request '${mergeRequest.title}' is ready to be merged!`);