Skip to content

Commit 9942d0d

Browse files
committed
improve debugging logs based on PR feedback
1 parent 671ece8 commit 9942d0d

4 files changed

Lines changed: 17 additions & 0 deletions

File tree

.github/actions/javascript/createOrUpdateStagingDeploy/index.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11953,6 +11953,10 @@ async function getPullRequestsDeployedBetween(fromTag, toTag) {
1195311953
core.startGroup('[api] Parsed PRs:');
1195411954
core.info(apiPullRequestNumbers.join(', '));
1195511955
core.endGroup();
11956+
// Print diff between git log and API results for debugging
11957+
const onlyInGitLog = gitLogPullRequestNumbers.filter((pr) => !apiPullRequestNumbers.includes(pr));
11958+
const onlyInAPI = apiPullRequestNumbers.filter((pr) => !gitLogPullRequestNumbers.includes(pr));
11959+
core.info(`PR list diff - git log only: [${onlyInGitLog.join(', ')}], API only: [${onlyInAPI.join(', ')}]`);
1195611960
return apiPullRequestNumbers;
1195711961
}
1195811962
exports["default"] = {

.github/actions/javascript/getDeployPullRequestList/index.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11991,6 +11991,10 @@ async function getPullRequestsDeployedBetween(fromTag, toTag) {
1199111991
core.startGroup('[api] Parsed PRs:');
1199211992
core.info(apiPullRequestNumbers.join(', '));
1199311993
core.endGroup();
11994+
// Print diff between git log and API results for debugging
11995+
const onlyInGitLog = gitLogPullRequestNumbers.filter((pr) => !apiPullRequestNumbers.includes(pr));
11996+
const onlyInAPI = apiPullRequestNumbers.filter((pr) => !gitLogPullRequestNumbers.includes(pr));
11997+
core.info(`PR list diff - git log only: [${onlyInGitLog.join(', ')}], API only: [${onlyInAPI.join(', ')}]`);
1199411998
return apiPullRequestNumbers;
1199511999
}
1199612000
exports["default"] = {

.github/actions/javascript/getPreviousVersion/index.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11829,6 +11829,10 @@ async function getPullRequestsDeployedBetween(fromTag, toTag) {
1182911829
core.startGroup('[api] Parsed PRs:');
1183011830
core.info(apiPullRequestNumbers.join(', '));
1183111831
core.endGroup();
11832+
// Print diff between git log and API results for debugging
11833+
const onlyInGitLog = gitLogPullRequestNumbers.filter((pr) => !apiPullRequestNumbers.includes(pr));
11834+
const onlyInAPI = apiPullRequestNumbers.filter((pr) => !gitLogPullRequestNumbers.includes(pr));
11835+
core.info(`PR list diff - git log only: [${onlyInGitLog.join(', ')}], API only: [${onlyInAPI.join(', ')}]`);
1183211836
return apiPullRequestNumbers;
1183311837
}
1183411838
exports["default"] = {

.github/libs/GitUtils.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,11 @@ async function getPullRequestsDeployedBetween(fromTag: string, toTag: string) {
211211
core.info(apiPullRequestNumbers.join(', '));
212212
core.endGroup();
213213

214+
// Print diff between git log and API results for debugging
215+
const onlyInGitLog = gitLogPullRequestNumbers.filter((pr) => !apiPullRequestNumbers.includes(pr));
216+
const onlyInAPI = apiPullRequestNumbers.filter((pr) => !gitLogPullRequestNumbers.includes(pr));
217+
core.info(`PR list diff - git log only: [${onlyInGitLog.join(', ')}], API only: [${onlyInAPI.join(', ')}]`);
218+
214219
return apiPullRequestNumbers;
215220
}
216221

0 commit comments

Comments
 (0)