Merge pull request #3771 from parasharrajat/gh-actions

[NO QA] Added CP checking for deployed PR notfication
Expensify · Jul 1, 2021 · 8e0c6f5 · 8e0c6f5
2 parents 78238db + f57d2ac
commit 8e0c6f5
Show file tree

Hide file tree

Showing 15 changed files with 6,285 additions and 192 deletions.
diff --git a/.github/actions/checkDeployBlockers/index.js b/.github/actions/checkDeployBlockers/index.js
@@ -350,6 +350,30 @@ class GithubUtils {
             ));
     }
 
+    /**
+     * Fetch all pull requests given a list of PR numbers.
+     *
+     * @param {Array<Number>} pullRequestNumbers
+     * @returns {Promise}
+     */
+    static fetchAllPullRequests(pullRequestNumbers) {
+        const oldestPR = _.first(_.sortBy(pullRequestNumbers));
+        return this.octokit.paginate(this.octokit.pulls.list, {
+            owner: GITHUB_OWNER,
+            repo: EXPENSIFY_CASH_REPO,
+            state: 'all',
+            sort: 'created',
+            direction: 'desc',
+            per_page: 100,
+        }, ({data}, done) => {
+            if (_.find(data, pr => pr.number === oldestPR)) {
+                done();
+            }
+            return data;
+        })
+            .then(prList => _.filter(prList, pr => _.contains(pullRequestNumbers, pr.number)));
+    }
+
     /**
      * Create comment on pull request
      *

diff --git a/.github/actions/createOrUpdateStagingDeploy/index.js b/.github/actions/createOrUpdateStagingDeploy/index.js
@@ -472,6 +472,30 @@ class GithubUtils {
             ));
     }
 
+    /**
+     * Fetch all pull requests given a list of PR numbers.
+     *
+     * @param {Array<Number>} pullRequestNumbers
+     * @returns {Promise}
+     */
+    static fetchAllPullRequests(pullRequestNumbers) {
+        const oldestPR = _.first(_.sortBy(pullRequestNumbers));
+        return this.octokit.paginate(this.octokit.pulls.list, {
+            owner: GITHUB_OWNER,
+            repo: EXPENSIFY_CASH_REPO,
+            state: 'all',
+            sort: 'created',
+            direction: 'desc',
+            per_page: 100,
+        }, ({data}, done) => {
+            if (_.find(data, pr => pr.number === oldestPR)) {
+                done();
+            }
+            return data;
+        })
+            .then(prList => _.filter(prList, pr => _.contains(pullRequestNumbers, pr.number)));
+    }
+
     /**
      * Create comment on pull request
      *

diff --git a/.github/actions/getMergeCommitForPullRequest/index.js b/.github/actions/getMergeCommitForPullRequest/index.js
@@ -378,6 +378,30 @@ class GithubUtils {
             ));
     }
 
+    /**
+     * Fetch all pull requests given a list of PR numbers.
+     *
+     * @param {Array<Number>} pullRequestNumbers
+     * @returns {Promise}
+     */
+    static fetchAllPullRequests(pullRequestNumbers) {
+        const oldestPR = _.first(_.sortBy(pullRequestNumbers));
+        return this.octokit.paginate(this.octokit.pulls.list, {
+            owner: GITHUB_OWNER,
+            repo: EXPENSIFY_CASH_REPO,
+            state: 'all',
+            sort: 'created',
+            direction: 'desc',
+            per_page: 100,
+        }, ({data}, done) => {
+            if (_.find(data, pr => pr.number === oldestPR)) {
+                done();
+            }
+            return data;
+        })
+            .then(prList => _.filter(prList, pr => _.contains(pullRequestNumbers, pr.number)));
+    }
+
     /**
      * Create comment on pull request
      *

diff --git a/.github/actions/getReleaseBody/index.js b/.github/actions/getReleaseBody/index.js
@@ -321,6 +321,30 @@ class GithubUtils {
             ));
     }
 
+    /**
+     * Fetch all pull requests given a list of PR numbers.
+     *
+     * @param {Array<Number>} pullRequestNumbers
+     * @returns {Promise}
+     */
+    static fetchAllPullRequests(pullRequestNumbers) {
+        const oldestPR = _.first(_.sortBy(pullRequestNumbers));
+        return this.octokit.paginate(this.octokit.pulls.list, {
+            owner: GITHUB_OWNER,
+            repo: EXPENSIFY_CASH_REPO,
+            state: 'all',
+            sort: 'created',
+            direction: 'desc',
+            per_page: 100,
+        }, ({data}, done) => {
+            if (_.find(data, pr => pr.number === oldestPR)) {
+                done();
+            }
+            return data;
+        })
+            .then(prList => _.filter(prList, pr => _.contains(pullRequestNumbers, pr.number)));
+    }
+
     /**
      * Create comment on pull request
      *

diff --git a/.github/actions/isPullRequestMergeable/index.js b/.github/actions/isPullRequestMergeable/index.js
@@ -324,6 +324,30 @@ class GithubUtils {
             ));
     }
 
+    /**
+     * Fetch all pull requests given a list of PR numbers.
+     *
+     * @param {Array<Number>} pullRequestNumbers
+     * @returns {Promise}
+     */
+    static fetchAllPullRequests(pullRequestNumbers) {
+        const oldestPR = _.first(_.sortBy(pullRequestNumbers));
+        return this.octokit.paginate(this.octokit.pulls.list, {
+            owner: GITHUB_OWNER,
+            repo: EXPENSIFY_CASH_REPO,
+            state: 'all',
+            sort: 'created',
+            direction: 'desc',
+            per_page: 100,
+        }, ({data}, done) => {
+            if (_.find(data, pr => pr.number === oldestPR)) {
+                done();
+            }
+            return data;
+        })
+            .then(prList => _.filter(prList, pr => _.contains(pullRequestNumbers, pr.number)));
+    }
+
     /**
      * Create comment on pull request
      *

diff --git a/.github/actions/isStagingDeployLocked/action.yml b/.github/actions/isStagingDeployLocked/action.yml
@@ -7,6 +7,8 @@ inputs:
 outputs:
     IS_LOCKED:
         description: Whether or not the open StagingDeployCash issue is locked.
+    NUMBER:
+        description: StagingDeployCash issue number
 runs:
     using: 'node12'
     main: 'index.js'
diff --git a/.github/actions/isStagingDeployLocked/index.js b/.github/actions/isStagingDeployLocked/index.js
@@ -14,13 +14,15 @@ const GithubUtils = __nccwpck_require__(7999);
 
 const run = function () {
     return GithubUtils.getStagingDeployCash()
-        .then(({labels}) => {
+        .then(({labels, number}) => {
             console.log(`Found StagingDeployCash with labels: ${_.pluck(labels, 'name')}`);
             core.setOutput('IS_LOCKED', _.contains(_.pluck(labels, 'name'), '🔐 LockCashDeploys 🔐'));
+            core.setOutput('NUMBER', number);
         })
         .catch((err) => {
             console.warn('No open StagingDeployCash found, continuing...', err);
             core.setOutput('IS_LOCKED', false);
+            core.setOutput('NUMBER', 0);
         });
 };
 
@@ -300,6 +302,30 @@ class GithubUtils {
             ));
     }
 
+    /**
+     * Fetch all pull requests given a list of PR numbers.
+     *
+     * @param {Array<Number>} pullRequestNumbers
+     * @returns {Promise}
+     */
+    static fetchAllPullRequests(pullRequestNumbers) {
+        const oldestPR = _.first(_.sortBy(pullRequestNumbers));
+        return this.octokit.paginate(this.octokit.pulls.list, {
+            owner: GITHUB_OWNER,
+            repo: EXPENSIFY_CASH_REPO,
+            state: 'all',
+            sort: 'created',
+            direction: 'desc',
+            per_page: 100,
+        }, ({data}, done) => {
+            if (_.find(data, pr => pr.number === oldestPR)) {
+                done();
+            }
+            return data;
+        })
+            .then(prList => _.filter(prList, pr => _.contains(pullRequestNumbers, pr.number)));
+    }
+
     /**
      * Create comment on pull request
      *

diff --git a/.github/actions/isStagingDeployLocked/isStagingDeployLocked.js b/.github/actions/isStagingDeployLocked/isStagingDeployLocked.js
@@ -4,13 +4,15 @@ const GithubUtils = require('../../libs/GithubUtils');
 
 const run = function () {
     return GithubUtils.getStagingDeployCash()
-        .then(({labels}) => {
+        .then(({labels, number}) => {
             console.log(`Found StagingDeployCash with labels: ${_.pluck(labels, 'name')}`);
             core.setOutput('IS_LOCKED', _.contains(_.pluck(labels, 'name'), '🔐 LockCashDeploys 🔐'));
+            core.setOutput('NUMBER', number);
         })
         .catch((err) => {
             console.warn('No open StagingDeployCash found, continuing...', err);
             core.setOutput('IS_LOCKED', false);
+            core.setOutput('NUMBER', 0);
         });
 };
 

diff --git a/.github/actions/markPullRequestsAsDeployed/action.yml b/.github/actions/markPullRequestsAsDeployed/action.yml
@@ -8,10 +8,9 @@ inputs:
         description: "Check if deploying to production"
         required: false
         default: "false"
-    IS_CHERRY_PICK:
-        description: "Did this deploy happen as part of a cherry-pick?"
-        required: false
-        default: "false"
+    STAGING_DEPLOY_NUMBER:
+        description: "StagingDeployCash issue number"
+        required: true
     DEPLOY_VERSION:
         description: "The app version in which the pull requests were deployed"
         required: true