Skip to content
Snippets Groups Projects
Commit 9732f3c7 authored by Rhys Arkins's avatar Rhys Arkins Committed by GitHub
Browse files

refactor: log assignees and reviewers success after completion (#999)

parent 709a8c20
No related branches found
No related tags found
No related merge requests found
...@@ -197,27 +197,33 @@ async function ensurePr(prConfig) { ...@@ -197,27 +197,33 @@ async function ensurePr(prConfig) {
async function addAssigneesReviewers(config, pr) { async function addAssigneesReviewers(config, pr) {
const { logger } = config; const { logger } = config;
if (config.assignees.length > 0) { if (config.assignees.length > 0) {
logger.debug({ assignees: config.assignees }, 'Adding assignees');
try { try {
const assignees = config.assignees.map( const assignees = config.assignees.map(
assignee => assignee =>
assignee.length && assignee[0] === '@' ? assignee.slice(1) : assignee assignee.length && assignee[0] === '@' ? assignee.slice(1) : assignee
); );
await config.api.addAssignees(pr.number, assignees); await config.api.addAssignees(pr.number, assignees);
logger.info({ assignees: config.assignees }, 'Added assignees');
} catch (err) { } catch (err) {
logger.warn({ err }, 'Failed to add assignees'); logger.info(
{ assignees: config.assignees, err },
'Failed to add assignees'
);
} }
} }
if (config.reviewers.length > 0) { if (config.reviewers.length > 0) {
logger.info({ reviewers: config.reviewers }, 'Adding reviewers');
try { try {
const reviewers = config.reviewers.map( const reviewers = config.reviewers.map(
reviewer => reviewer =>
reviewer.length && reviewer[0] === '@' ? reviewer.slice(1) : reviewer reviewer.length && reviewer[0] === '@' ? reviewer.slice(1) : reviewer
); );
await config.api.addReviewers(pr.number, reviewers); await config.api.addReviewers(pr.number, reviewers);
logger.info({ reviewers: config.reviewers }, 'Added reviewers');
} catch (err) { } catch (err) {
logger.warn({ err }, 'Failed to add reviewers'); logger.info(
{ assignees: config.assignees, err },
'Failed to add reviewers'
);
} }
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment