replace event trigger by refs to avoid duplicate runs

1 files changed, 44 insertions, 4 deletions

diff --git a/.drone.star b/.drone.star
index f438842f6..d74e6fd64 100644
--- a/.drone.star
+++ b/.drone.star
@@ -10,10 +10,10 @@ def main(ctx):
         "cron": ["translations-2-7"],
     }
     build_trigger = {
-        "event": [
-            "push",
-            "pull_request",
-            "tag",
+        "ref": [
+            "refs/heads/master",
+            "refs/tags/**",
+            "refs/pull/**",
         ],
     }
     changelog_trigger = {
@@ -23,6 +23,11 @@ def main(ctx):
         ],
     }
     pipelines = [
+        # Check starlark
+        check_starlark(
+            ctx,
+            build_trigger,
+        ),
         # Build changelog
         changelog(
             ctx,
@@ -48,7 +53,9 @@ def main(ctx):
         ),
         notification(
             name = "build",
+            trigger = build_trigger,
             depends_on = [
+                "check-starlark",
                 "changelog",
                 "gcc-release-make",
                 "clang-debug-ninja",
@@ -101,6 +108,39 @@ def from_secret(name):
         "from_secret": name,
     }
 
+def check_starlark(ctx, trigger = {}, depends_on = []):
+    return {
+        "kind": "pipeline",
+        "type": "docker",
+        "name": "check-starlark",
+        "steps": [
+            {
+                "name": "format-check-starlark",
+                "image": "owncloudci/bazel-buildifier",
+                "pull": "always",
+                "commands": [
+                    "buildifier --mode=check .drone.star",
+                ],
+            },
+            {
+                "name": "show-diff",
+                "image": "owncloudci/bazel-buildifier",
+                "pull": "always",
+                "commands": [
+                    "buildifier --mode=fix .drone.star",
+                    "git diff",
+                ],
+                "when": {
+                    "status": [
+                        "failure",
+                    ],
+                },
+            },
+        ],
+        "depends_on": depends_on,
+        "trigger": trigger,
+    }
+
 def build_and_test_client(ctx, c_compiler, cxx_compiler, build_type, generator, trigger = {}, depends_on = []):
     build_command = "ninja" if generator == "Ninja" else "make"
     pipeline_name = c_compiler + "-" + build_type.lower() + "-" + build_command