Repo gardening: don't send slack notifs for closed issues (#41165)

Committed via a GitHub action: https://github.com/Automattic/jetpack/actions/runs/12905715134 Upstream-Ref: Automattic/jetpack@dbbc085
Automattic · Jan 22, 2025 · 6d55f28 · 6d55f28
1 parent 83fd36f
commit 6d55f28
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -50137,7 +50137,7 @@ async function createOrUpdateComment( payload, octokit, issueReferences, issueCo
  */
 async function addHappinessLabel( payload, octokit ) {
 	const {
-		issue: { number },
+		issue: { number, state },
 		repository: {
 			name: repo,
 			owner: { login: ownerLogin },
@@ -50164,9 +50164,10 @@ async function addHappinessLabel( payload, octokit ) {
 
 	// Send Slack notification, if we have the necessary tokens.
 	// No Slack tokens, we won't be able to escalate. Bail.
+	// If the issue is already closed, do not send any Slack reminder.
 	const slackToken = getInput( 'slack_token' );
 	const channel = getInput( 'slack_quality_channel' );
-	if ( ! slackToken || ! channel ) {
+	if ( ! slackToken || ! channel || state === 'closed' ) {
 		return false;
 	}
 

diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/tasks/gather-support-references/index.js b/src/tasks/gather-support-references/index.js
@@ -414,7 +414,7 @@ async function createOrUpdateComment( payload, octokit, issueReferences, issueCo
  */
 async function addHappinessLabel( payload, octokit ) {
 	const {
-		issue: { number },
+		issue: { number, state },
 		repository: {
 			name: repo,
 			owner: { login: ownerLogin },
@@ -441,9 +441,10 @@ async function addHappinessLabel( payload, octokit ) {
 
 	// Send Slack notification, if we have the necessary tokens.
 	// No Slack tokens, we won't be able to escalate. Bail.
+	// If the issue is already closed, do not send any Slack reminder.
 	const slackToken = getInput( 'slack_token' );
 	const channel = getInput( 'slack_quality_channel' );
-	if ( ! slackToken || ! channel ) {
+	if ( ! slackToken || ! channel || state === 'closed' ) {
 		return false;
 	}