Don't test pull requests
diff mbox series

Message ID 20180724100203.27481-1-andrew.donnellan@au1.ibm.com
State Accepted
Headers show
Series
  • Don't test pull requests
Related show

Commit Message

Andrew Donnellan July 24, 2018, 10:02 a.m. UTC
Testing pull request emails is a bit silly. Skip "patches" that have a pull
URL.

Closes: #35 ("Don't test pull requests")
Signed-off-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>

---

untested, but yolo, should work
---
 src/main.rs | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Russell Currey July 31, 2018, 11:25 p.m. UTC | #1
On Tue, 2018-07-24 at 20:02 +1000, Andrew Donnellan wrote:
> Testing pull request emails is a bit silly. Skip "patches" that have
> a pull
> URL.
> 
> Closes: #35 ("Don't test pull requests")
> Signed-off-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
> 
Thanks, merged as 09a23c9505afd0e80366d5be7609f2641a85e88c

Patch
diff mbox series

diff --git a/src/main.rs b/src/main.rs
index eb7ffc1ee36d..776aea299a79 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -429,6 +429,12 @@  fn main() {
                 continue;
             }
 
+            // Skip pull requests
+            if patch.pull_url.is_some() {
+                debug!("Skipping pull request {}", patch.name);
+                continue;
+            }
+
             // Skip if it's the wrong project
             if patch.project.link_name != args.flag_project {
                 warn!(