mikepenz/release-changelog-builder-action
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

- introduce more lgos to better identify when PRs are filtered out

Browse Source 
- Related to: https://github.com/mikepenz/release-changelog-builder-action/issues/939
This commit is contained in:
Mike Penz
2022-10-27 17:02:32 +02:00
parent d77f10180b
commit 2800a9dd18
3 changed files with 15 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
dist/index.js
Generated Vendored
+5 -1
View File
@@ -832,7 +832,7 @@ class ReleaseNotes {
core.info(`ℹ️ Fetching PRs between dates ${fromDate.toISOString()} to ${toDate.toISOString()} for ${owner}/${repo}`); core.info(`ℹ️ Fetching PRs between dates ${fromDate.toISOString()} to ${toDate.toISOString()} for ${owner}/${repo}`);
const pullRequestsApi = new pullRequests_1.PullRequests(octokit); const pullRequestsApi = new pullRequests_1.PullRequests(octokit);
const pullRequests = yield pullRequestsApi.getBetweenDates(owner, repo, fromDate, toDate, configuration.max_pull_requests || configuration_1.DefaultConfiguration.max_pull_requests); const pullRequests = yield pullRequestsApi.getBetweenDates(owner, repo, fromDate, toDate, configuration.max_pull_requests || configuration_1.DefaultConfiguration.max_pull_requests);
core.info(`ℹ️ Retrieved ${pullRequests.length} merged PRs for ${owner}/${repo}`); core.info(`ℹ️ Retrieved ${pullRequests.length} PRs for ${owner}/${repo} in date range from API`);
const prCommits = (0, commits_1.filterCommits)(commits, configuration.exclude_merge_branches || configuration_1.DefaultConfiguration.exclude_merge_branches); const prCommits = (0, commits_1.filterCommits)(commits, configuration.exclude_merge_branches || configuration_1.DefaultConfiguration.exclude_merge_branches);
core.info(`ℹ️ Retrieved ${prCommits.length} release commits for ${owner}/${repo}`); core.info(`ℹ️ Retrieved ${prCommits.length} release commits for ${owner}/${repo}`);
// create array of commits for this release // create array of commits for this release
@@ -843,6 +843,7 @@ class ReleaseNotes {
const mergedPullRequests = pullRequests.filter(pr => { const mergedPullRequests = pullRequests.filter(pr => {
return releaseCommitHashes.includes(pr.mergeCommitSha); return releaseCommitHashes.includes(pr.mergeCommitSha);
}); });
core.info(`ℹ️ Retrieved ${mergedPullRequests.length} merged PRs for ${owner}/${repo}`);
let allPullRequests = mergedPullRequests; let allPullRequests = mergedPullRequests;
if (includeOpen) { if (includeOpen) {
// retrieve all open pull requests // retrieve all open pull requests
@@ -866,6 +867,9 @@ class ReleaseNotes {
} }
return true; return true;
}); });
if (baseBranches.length !== 0) {
core.info(`ℹ️ Retrieved ${mergedPullRequests.length} PRs for ${owner}/${repo} filtered by the 'base_branches' configuration.`);
}
if (fetchReviewers) { if (fetchReviewers) {
core.info(`ℹ️ Fetching reviewers was enabled`); core.info(`ℹ️ Fetching reviewers was enabled`);
// update PR information with reviewers who approved // update PR information with reviewers who approved
dist/index.js.map
Generated Vendored
+1 -1
View File
File diff suppressed because one or more lines are too long
src/releaseNotes.ts
+9 -1
View File
@@ -125,7 +125,7 @@ export class ReleaseNotes {
configuration.max_pull_requests || DefaultConfiguration.max_pull_requests configuration.max_pull_requests || DefaultConfiguration.max_pull_requests
) )
core.info(`ℹ️ Retrieved ${pullRequests.length} merged PRs for ${owner}/${repo}`) core.info(`ℹ️ Retrieved ${pullRequests.length} PRs for ${owner}/${repo} in date range from API`)
const prCommits = filterCommits( const prCommits = filterCommits(
commits, commits,
@@ -144,6 +144,8 @@ export class ReleaseNotes {
return releaseCommitHashes.includes(pr.mergeCommitSha) return releaseCommitHashes.includes(pr.mergeCommitSha)
}) })
core.info(`ℹ️ Retrieved ${mergedPullRequests.length} merged PRs for ${owner}/${repo}`)
let allPullRequests = mergedPullRequests let allPullRequests = mergedPullRequests
if (includeOpen) { if (includeOpen) {
// retrieve all open pull requests // retrieve all open pull requests
@@ -177,6 +179,12 @@ export class ReleaseNotes {
return true return true
}) })
if (baseBranches.length !== 0) {
core.info(
`ℹ️ Retrieved ${mergedPullRequests.length} PRs for ${owner}/${repo} filtered by the 'base_branches' configuration.`
)
}
if (fetchReviewers) { if (fetchReviewers) {
core.info(`ℹ️ Fetching reviewers was enabled`) core.info(`ℹ️ Fetching reviewers was enabled`)
// update PR information with reviewers who approved // update PR information with reviewers who approved