Skip to content

Commit 660fce0

Browse files
committed
Fix failure with existing issue
1 parent e5e8e8e commit 660fce0

File tree

1 file changed

+21
-13
lines changed

1 file changed

+21
-13
lines changed

src/action.js

Lines changed: 21 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -116,20 +116,28 @@ const triage = async(componentsToIssueNumber, componentsToIssueMetadata, compone
116116
const error = componentsToPlanState[slug].error;
117117

118118
if (drifted || error) {
119-
const commitSHA = componentsToIssueMetadata[slug].commitSHA;
120-
const currentSHA = "${{ github.sha }}";
121-
if (currentSHA === commitSHA) {
122-
core.info(`Component "${slug}" marked as drifted but default branch SHA didn't change so nothing to update. Skipping ...`);
123-
driftingComponents.push(slug);
119+
const commitSHA = componentsToIssueMetadata[slug].commitSHA;
120+
const currentSHA = "${{ github.sha }}";
121+
if (currentSHA === commitSHA) {
122+
core.info(`Component "${slug}" marked as drifted but default branch SHA didn't change so nothing to update. Skipping ...`);
123+
if (error) {
124+
erroredComponents.push(slug)
125+
} else {
126+
driftingComponents.push(slug);
127+
}
128+
} else {
129+
core.info(`Component "${slug}" is still drifting. Issue ${issueNumber} needs to be updated.`);
130+
componentsToUpdateExistingIssue.push(slug);
131+
if (error) {
132+
erroredComponents.push(slug)
133+
} else {
134+
driftingComponents.push(slug);
135+
}
136+
}
124137
} else {
125-
core.info(`Component "${slug}" is still drifting. Issue ${issueNumber} needs to be updated.`);
126-
componentsToUpdateExistingIssue.push(slug);
127-
driftingComponents.push(slug);
128-
}
129-
} else {
130-
core.info(`Component "${slug}" is not drifting anymore. Issue ${issueNumber} needs to be closed.`);
131-
componentsCandidatesToCloseIssue.push(slug);
132-
recoveredComponents.push(slug);
138+
core.info(`Component "${slug}" is not drifting anymore. Issue ${issueNumber} needs to be closed.`);
139+
componentsCandidatesToCloseIssue.push(slug);
140+
recoveredComponents.push(slug);
133141
}
134142
} else {
135143
core.info(`Component "${slug}" has been removed. Issue ${issueNumber} needs to be closed.`);

0 commit comments

Comments
 (0)