Merge pull request #3913 from Expensify/Rory-LogGetPRsMergedBetween

[No QA] Add logs directly inside GetPRsMergedBetween
Expensify · Jul 7, 2021 · dd90800 · dd90800
2 parents f11d808 + 361d317
commit dd90800
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/.github/actions/createOrUpdateStagingDeploy/index.js b/.github/actions/createOrUpdateStagingDeploy/index.js
@@ -193,6 +193,7 @@ const {execSync} = __nccwpck_require__(3129);
  * @returns {Array}
  */
 function getPullRequestsMergedBetween(fromRef, toRef) {
+    console.log('Getting pull requests merged between the following refs:', fromRef, toRef);
     const localGitLogs = execSync(`git log --format="%s" ${fromRef}...${toRef}`).toString();
     return _.map(
         [...localGitLogs.matchAll(/Merge pull request #(\d{1,6}) from (?!Expensify\/(?:master|main|version-))/g)],

diff --git a/.github/actions/getDeployPullRequestList/index.js b/.github/actions/getDeployPullRequestList/index.js
@@ -117,6 +117,7 @@ const {execSync} = __nccwpck_require__(3129);
  * @returns {Array}
  */
 function getPullRequestsMergedBetween(fromRef, toRef) {
+    console.log('Getting pull requests merged between the following refs:', fromRef, toRef);
     const localGitLogs = execSync(`git log --format="%s" ${fromRef}...${toRef}`).toString();
     return _.map(
         [...localGitLogs.matchAll(/Merge pull request #(\d{1,6}) from (?!Expensify\/(?:master|main|version-))/g)],

diff --git a/.github/libs/GitUtils.js b/.github/libs/GitUtils.js
@@ -9,6 +9,7 @@ const {execSync} = require('child_process');
  * @returns {Array}
  */
 function getPullRequestsMergedBetween(fromRef, toRef) {
+    console.log('Getting pull requests merged between the following refs:', fromRef, toRef);
     const localGitLogs = execSync(`git log --format="%s" ${fromRef}...${toRef}`).toString();
     return _.map(
         [...localGitLogs.matchAll(/Merge pull request #(\d{1,6}) from (?!Expensify\/(?:master|main|version-))/g)],