From dd81e1e3345b91c4cb4653b697c1054526f6b924 Mon Sep 17 00:00:00 2001 From: dundargoc Date: Sun, 12 Nov 2023 21:40:24 +0100 Subject: ci: refactor CI files Mostly rename file and variable names to be more consistent. This makes it easier to locate them in the "Actions" tab on github. --- .github/scripts/remove-reviewers.js | 16 ----- .github/scripts/reviewers_add.js | 118 ++++++++++++++++++++++++++++++++++++ .github/scripts/reviewers_remove.js | 16 +++++ .github/scripts/reviews.js | 118 ------------------------------------ 4 files changed, 134 insertions(+), 134 deletions(-) delete mode 100644 .github/scripts/remove-reviewers.js create mode 100644 .github/scripts/reviewers_add.js create mode 100644 .github/scripts/reviewers_remove.js delete mode 100644 .github/scripts/reviews.js (limited to '.github/scripts') diff --git a/.github/scripts/remove-reviewers.js b/.github/scripts/remove-reviewers.js deleted file mode 100644 index 9e44e4ac86..0000000000 --- a/.github/scripts/remove-reviewers.js +++ /dev/null @@ -1,16 +0,0 @@ -module.exports = async ({ github, context }) => { - const requestedReviewers = await github.rest.pulls.listRequestedReviewers({ - owner: context.repo.owner, - repo: context.repo.repo, - pull_number: context.issue.number, - }); - - const reviewers = requestedReviewers.data.users.map((e) => e.login); - - github.rest.pulls.removeRequestedReviewers({ - owner: context.repo.owner, - repo: context.repo.repo, - pull_number: context.issue.number, - reviewers: reviewers, - }); -}; diff --git a/.github/scripts/reviewers_add.js b/.github/scripts/reviewers_add.js new file mode 100644 index 0000000000..d28d91c2f6 --- /dev/null +++ b/.github/scripts/reviewers_add.js @@ -0,0 +1,118 @@ +module.exports = async ({ github, context }) => { + const pr_data = await github.rest.pulls.get({ + owner: context.repo.owner, + repo: context.repo.repo, + pull_number: context.issue.number, + }); + const labels = pr_data.data.labels.map((e) => e.name); + const reviewers = new Set(); + + if (labels.includes("api")) { + reviewers.add("bfredl"); + reviewers.add("famiu"); + } + + if (labels.includes("build")) { + reviewers.add("dundargoc"); + reviewers.add("jamessan"); + reviewers.add("justinmk"); + } + + if (labels.includes("ci")) { + reviewers.add("dundargoc"); + reviewers.add("jamessan"); + reviewers.add("justinmk"); + } + + if (labels.includes("column")) { + reviewers.add("lewis6991"); + } + + if (labels.includes("dependencies")) { + reviewers.add("jamessan"); + } + + if (labels.includes("diagnostic")) { + reviewers.add("gpanders"); + } + + if (labels.includes("diff")) { + reviewers.add("lewis6991"); + } + + if (labels.includes("distribution")) { + reviewers.add("jamessan"); + } + + if (labels.includes("documentation")) { + reviewers.add("clason"); + } + + if (labels.includes("extmarks")) { + reviewers.add("bfredl"); + } + + if (labels.includes("filetype")) { + reviewers.add("clason"); + reviewers.add("gpanders"); + reviewers.add("smjonas"); + } + + if (labels.includes("lsp")) { + reviewers.add("folke"); + reviewers.add("MariaSolOs"); + reviewers.add("mfussenegger"); + } + + if (labels.includes("options")) { + reviewers.add("famiu"); + } + + if (labels.includes("platform:nix")) { + reviewers.add("teto"); + } + + if (labels.includes("project-management")) { + reviewers.add("bfredl"); + reviewers.add("justinmk"); + } + + if (labels.includes("statusline")) { + reviewers.add("famiu"); + } + + if (labels.includes("test")) { + reviewers.add("justinmk"); + } + + if (labels.includes("treesitter")) { + reviewers.add("bfredl"); + reviewers.add("clason"); + reviewers.add("lewis6991"); + } + + if (labels.includes("typo")) { + reviewers.add("dundargoc"); + } + + if (labels.includes("ui")) { + reviewers.add("bfredl"); + reviewers.add("famiu"); + } + + if (labels.includes("vim-patch")) { + reviewers.add("seandewar"); + reviewers.add("zeertzjq"); + } + + // Remove person that opened the PR since they can't review themselves + const pr_opener = pr_data.data.user.login; + reviewers.delete(pr_opener); + + github.rest.pulls.requestReviewers({ + owner: context.repo.owner, + repo: context.repo.repo, + pull_number: context.issue.number, + reviewers: Array.from(reviewers), + }); +}; diff --git a/.github/scripts/reviewers_remove.js b/.github/scripts/reviewers_remove.js new file mode 100644 index 0000000000..9e44e4ac86 --- /dev/null +++ b/.github/scripts/reviewers_remove.js @@ -0,0 +1,16 @@ +module.exports = async ({ github, context }) => { + const requestedReviewers = await github.rest.pulls.listRequestedReviewers({ + owner: context.repo.owner, + repo: context.repo.repo, + pull_number: context.issue.number, + }); + + const reviewers = requestedReviewers.data.users.map((e) => e.login); + + github.rest.pulls.removeRequestedReviewers({ + owner: context.repo.owner, + repo: context.repo.repo, + pull_number: context.issue.number, + reviewers: reviewers, + }); +}; diff --git a/.github/scripts/reviews.js b/.github/scripts/reviews.js deleted file mode 100644 index d28d91c2f6..0000000000 --- a/.github/scripts/reviews.js +++ /dev/null @@ -1,118 +0,0 @@ -module.exports = async ({ github, context }) => { - const pr_data = await github.rest.pulls.get({ - owner: context.repo.owner, - repo: context.repo.repo, - pull_number: context.issue.number, - }); - const labels = pr_data.data.labels.map((e) => e.name); - const reviewers = new Set(); - - if (labels.includes("api")) { - reviewers.add("bfredl"); - reviewers.add("famiu"); - } - - if (labels.includes("build")) { - reviewers.add("dundargoc"); - reviewers.add("jamessan"); - reviewers.add("justinmk"); - } - - if (labels.includes("ci")) { - reviewers.add("dundargoc"); - reviewers.add("jamessan"); - reviewers.add("justinmk"); - } - - if (labels.includes("column")) { - reviewers.add("lewis6991"); - } - - if (labels.includes("dependencies")) { - reviewers.add("jamessan"); - } - - if (labels.includes("diagnostic")) { - reviewers.add("gpanders"); - } - - if (labels.includes("diff")) { - reviewers.add("lewis6991"); - } - - if (labels.includes("distribution")) { - reviewers.add("jamessan"); - } - - if (labels.includes("documentation")) { - reviewers.add("clason"); - } - - if (labels.includes("extmarks")) { - reviewers.add("bfredl"); - } - - if (labels.includes("filetype")) { - reviewers.add("clason"); - reviewers.add("gpanders"); - reviewers.add("smjonas"); - } - - if (labels.includes("lsp")) { - reviewers.add("folke"); - reviewers.add("MariaSolOs"); - reviewers.add("mfussenegger"); - } - - if (labels.includes("options")) { - reviewers.add("famiu"); - } - - if (labels.includes("platform:nix")) { - reviewers.add("teto"); - } - - if (labels.includes("project-management")) { - reviewers.add("bfredl"); - reviewers.add("justinmk"); - } - - if (labels.includes("statusline")) { - reviewers.add("famiu"); - } - - if (labels.includes("test")) { - reviewers.add("justinmk"); - } - - if (labels.includes("treesitter")) { - reviewers.add("bfredl"); - reviewers.add("clason"); - reviewers.add("lewis6991"); - } - - if (labels.includes("typo")) { - reviewers.add("dundargoc"); - } - - if (labels.includes("ui")) { - reviewers.add("bfredl"); - reviewers.add("famiu"); - } - - if (labels.includes("vim-patch")) { - reviewers.add("seandewar"); - reviewers.add("zeertzjq"); - } - - // Remove person that opened the PR since they can't review themselves - const pr_opener = pr_data.data.user.login; - reviewers.delete(pr_opener); - - github.rest.pulls.requestReviewers({ - owner: context.repo.owner, - repo: context.repo.repo, - pull_number: context.issue.number, - reviewers: Array.from(reviewers), - }); -}; -- cgit