This is an automated email from the ASF dual-hosted git repository. wangzx pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/echarts-bot.git
The following commit(s) were added to refs/heads/master by this push: new c186fc8 fix: don't add `awaiting review` if the PR is draft. c186fc8 is described below commit c186fc8f670489c883a4ea6cc3d30b85167b5256 Author: plainheart <y...@all-my-life.cn> AuthorDate: Mon Mar 29 10:50:30 2021 +0800 fix: don't add `awaiting review` if the PR is draft. --- index.js | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/index.js b/index.js index dbf2913..5ba0e4a 100644 --- a/index.js +++ b/index.js @@ -190,13 +190,16 @@ module.exports = (app) => { }); app.on(['pull_request.synchronize'], async context => { - const addLabel = context.octokit.issues.addLabels( - context.issue({ - labels: ['PR: awaiting review'] - }) - ); + let addLabel; const removeLabel = getRemoveLabel(context, 'PR: revision needed'); - return Promise.all([addLabel, removeLabel]); + if (!context.payload.pull_request.draft) { + addLabel = context.octokit.issues.addLabels( + context.issue({ + labels: ['PR: awaiting review'] + }) + ); + } + return Promise.all([removeLabel, addLabel]); }); app.on(['pull_request.closed'], async context => { --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@echarts.apache.org For additional commands, e-mail: commits-h...@echarts.apache.org