Merge pull request #1844 from nikomatsakis/patch-1

jackh726 · web-flow · commit f563bc9578ae · 2024-09-17T10:12:35.000-04:00
only update zulip when tracking issues are opened
diff --git a/src/handlers/project_goals.rs b/src/handlers/project_goals.rs
@@ -226,7 +226,7 @@ pub async fn handle(ctx: &Context, event: &Event) -> anyhow::Result<()> {
             issue,
             ..
         }) => {
-            if issue.labels.iter().any(|l| l.name == C_TRACKING_ISSUE) {
+            if !issue.labels.iter().any(|l| l.name == C_TRACKING_ISSUE) {
                 return Ok(());
             }
             let zulip_topic_name = zulip_topic_name(issue);

Original file line number	Diff line number	Diff line change
`@@ -226,7 +226,7 @@ pub async fn handle(ctx: &Context, event: &Event) -> anyhow::Result<()> {`
`226`	`226`	`issue,`
`227`	`227`	`..`
`228`	`228`	`}) => {`
`229`		`- if issue.labels.iter().any(\|l\| l.name == C_TRACKING_ISSUE) {`
	`229`	`+ if !issue.labels.iter().any(\|l\| l.name == C_TRACKING_ISSUE) {`
`230`	`230`	`return Ok(());`
`231`	`231`	`}`
`232`	`232`	`let zulip_topic_name = zulip_topic_name(issue);`