Skip to content

Commit 1e89d74

Browse files
committed
bugfix: fix for issue being created/updated after release
1 parent a5ca3c0 commit 1e89d74

File tree

3 files changed

+12
-10
lines changed

3 files changed

+12
-10
lines changed

dist/index.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6036,21 +6036,21 @@ async function runAction(token, staleDays, commitMessageLines) {
60366036
return logInfo('Could not find latest release');
60376037
}
60386038

6039-
logInfo(`Latest release - name:${latestRelease.name}, created:${latestRelease.created_at},
6039+
logInfo(`Latest release - name:${latestRelease.name}, published:${latestRelease.published_at},
60406040
Tag:${latestRelease.tag_name}, author:${latestRelease.author.login}`);
60416041

60426042
let pendingIssue = await getLastOpenPendingIssue(token);
60436043
const unreleasedCommits = await getUnreleasedCommits(
60446044
token,
6045-
latestRelease.created_at,
6045+
latestRelease.published_at,
60466046
staleDays,
60476047
);
60486048

60496049
if (unreleasedCommits.length) {
60506050
await createOrUpdateIssue(token, unreleasedCommits, pendingIssue, latestRelease, commitMessageLines);
60516051
} else {
60526052
logInfo('No pending commits found');
6053-
if (pendingIssue && Date.parse(latestRelease.created_at) > Date.parse(pendingIssue.updated_at)) {
6053+
if (pendingIssue && Date.parse(latestRelease.published_at) > Date.parse(pendingIssue.updated_at)) {
60546054
await closeIssue(token, pendingIssue.number);
60556055
}
60566056
}
@@ -6074,12 +6074,12 @@ async function getLatestRelease(token) {
60746074
const octokit = github.getOctokit(token);
60756075
const { owner, repo } = github.context.repo;
60766076

6077-
const allReleasesResp = await octokit.request(`GET /repos/{owner}/{repo}/releases/latest`, {
6077+
const latestReleaseResponse = await octokit.request(`GET /repos/{owner}/{repo}/releases/latest`, {
60786078
owner,
60796079
repo,
60806080
});
60816081

6082-
return allReleasesResp.data;
6082+
return latestReleaseResponse.data;
60836083
}
60846084

60856085
async function getUnreleasedCommits(token, latestReleaseDate, staleDays) {

src/release-notify-action.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,21 +11,21 @@ async function runAction(token, staleDays, commitMessageLines) {
1111
return logInfo('Could not find latest release');
1212
}
1313

14-
logInfo(`Latest release - name:${latestRelease.name}, created:${latestRelease.created_at},
14+
logInfo(`Latest release - name:${latestRelease.name}, published:${latestRelease.published_at},
1515
Tag:${latestRelease.tag_name}, author:${latestRelease.author.login}`);
1616

1717
let pendingIssue = await getLastOpenPendingIssue(token);
1818
const unreleasedCommits = await getUnreleasedCommits(
1919
token,
20-
latestRelease.created_at,
20+
latestRelease.published_at,
2121
staleDays,
2222
);
2323

2424
if (unreleasedCommits.length) {
2525
await createOrUpdateIssue(token, unreleasedCommits, pendingIssue, latestRelease, commitMessageLines);
2626
} else {
2727
logInfo('No pending commits found');
28-
if (pendingIssue && Date.parse(latestRelease.created_at) > Date.parse(pendingIssue.updated_at)) {
28+
if (pendingIssue && Date.parse(latestRelease.published_at) > Date.parse(pendingIssue.updated_at)) {
2929
await closeIssue(token, pendingIssue.number);
3030
}
3131
}

test/testData.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,13 +144,15 @@ const allReleasesData = {
144144
data: [
145145
{
146146
name: 'Release 2.0',
147-
created_at: '2021-04-27T07:37:09Z',
147+
created_at: '2021-03-27T07:37:09Z',
148+
published_at: '2021-04-27T07:37:09Z',
148149
Tag: '2.0',
149150
author: 'sameer'
150151
},
151152
{
152153
name: 'Release 1.0',
153-
created_at: '2021-04-20T07:37:09Z',
154+
created_at: '2021-03-20T07:37:09Z',
155+
published_at: '2021-04-20T07:37:09Z',
154156
Tag: '1.0',
155157
author: 'gilach'
156158
},

0 commit comments

Comments
 (0)