Quellcode durchsuchen

Merge pull request #1755 from jyn514/pr-edit

don't try to reassign a reviewer when a pr title is edited
Eric Huss vor 1 Jahr
Ursprung
Commit
413f9d421b
1 geänderte Dateien mit 11 neuen und 1 gelöschten Zeilen
  1. 11 1
      src/handlers.rs

+ 11 - 1
src/handlers.rs

@@ -180,7 +180,17 @@ macro_rules! command_handlers {
             errors: &mut Vec<HandlerError>,
         ) {
             match event {
-                Event::Issue(e) => if !matches!(e.action, IssuesAction::Opened | IssuesAction::Edited) {
+                // always handle new PRs / issues
+                Event::Issue(IssuesEvent { action: IssuesAction::Opened, .. }) => {},
+                Event::Issue(IssuesEvent { action: IssuesAction::Edited, .. }) => {
+                    // if the issue was edited, but we don't get a `changes[body]` diff, it means only the title was edited, not the body.
+                    // don't process the same commands twice.
+                    if event.comment_from().is_none() {
+                        log::debug!("skipping title-only edit event");
+                        return;
+                    }
+                },
+                Event::Issue(e) => {
                     // no change in issue's body for these events, so skip
                     log::debug!("skipping event, issue was {:?}", e.action);
                     return;