From fc475fda153b67fa6c97d48f0bf303babdc82db1 Mon Sep 17 00:00:00 2001
From: Dori Medini <dori@starkware.co>
Date: Wed, 24 Jul 2024 10:33:41 +0300
Subject: [PATCH] chore: improve blockifier CI trigger and scope

Signed-off-by: Dori Medini <dori@starkware.co>
---
 .github/workflows/blockifier_ci.yml             | 14 ++++++++++++++
 .github/workflows/blockifier_compiled_cairo.yml |  6 +++++-
 .github/workflows/blockifier_coverage.yml       |  4 ++++
 .github/workflows/blockifier_post-merge.yml     |  6 +++++-
 4 files changed, 28 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/blockifier_ci.yml b/.github/workflows/blockifier_ci.yml
index 2a4c1d1a1e..a638addb8d 100644
--- a/.github/workflows/blockifier_ci.yml
+++ b/.github/workflows/blockifier_ci.yml
@@ -8,7 +8,14 @@ on:
     tags:
       - v[0-9].**
     paths:
+      - '.github/workflows/blockifier_ci.yml'
+      - 'Cargo.toml'
+      - 'Cargo.lock'
+      - 'blockifier.Dockerfile'
+      - 'build_native_blockifier.sh'
       - 'crates/blockifier/**'
+      - 'crates/native_blockifier/**'
+      - 'scripts/install_build_tools.sh'
 
   pull_request:
     types:
@@ -18,7 +25,14 @@ on:
       - auto_merge_enabled
       - edited
     paths:
+      - '.github/workflows/blockifier_ci.yml'
+      - 'Cargo.toml'
+      - 'Cargo.lock'
+      - 'blockifier.Dockerfile'
+      - 'build_native_blockifier.sh'
       - 'crates/blockifier/**'
+      - 'crates/native_blockifier/**'
+      - 'scripts/install_build_tools.sh'
 
 jobs:
   featureless-build:
diff --git a/.github/workflows/blockifier_compiled_cairo.yml b/.github/workflows/blockifier_compiled_cairo.yml
index c4111b1118..1de8b79d18 100644
--- a/.github/workflows/blockifier_compiled_cairo.yml
+++ b/.github/workflows/blockifier_compiled_cairo.yml
@@ -7,14 +7,18 @@ on:
     tags:
     - v[0-9].**
     paths:
+    - '.github/workflows/blockifier_compiled_cairo.yml'
     - 'crates/blockifier/feature_contracts/cairo0/**'
+    - 'crates/blockifier/tests/requirements.txt'
   pull_request:
     types:
       - opened
       - reopened
       - synchronize
     paths:
+     - '.github/workflows/blockifier_compiled_cairo.yml'
      - 'crates/blockifier/feature_contracts/cairo0/**'
+     - 'crates/blockifier/tests/requirements.txt'
 
 jobs:
   verify_cairo_file_dependencies:
@@ -38,4 +42,4 @@ jobs:
 
       - run:
           pip install -r crates/blockifier/tests/requirements.txt;
-          cargo test verify_feature_contracts -- --include-ignored
+          cargo test -p blockifier verify_feature_contracts -- --include-ignored
diff --git a/.github/workflows/blockifier_coverage.yml b/.github/workflows/blockifier_coverage.yml
index fd5541d041..3ec8c79603 100644
--- a/.github/workflows/blockifier_coverage.yml
+++ b/.github/workflows/blockifier_coverage.yml
@@ -3,10 +3,14 @@ name: Blockifier-Coverage
 on:
   pull_request:
     paths:
+      - '.github/workflows/blockifier_coverage.yml'
       - 'crates/blockifier/**'
+      - 'crates/native_blockifier/**'
   push:
     paths:
+      - '.github/workflows/blockifier_coverage.yml'
       - 'crates/blockifier/**'
+      - 'crates/native_blockifier/**'
 
 jobs:
   coverage:
diff --git a/.github/workflows/blockifier_post-merge.yml b/.github/workflows/blockifier_post-merge.yml
index 4de0e4e9a9..b9bc6cd01e 100644
--- a/.github/workflows/blockifier_post-merge.yml
+++ b/.github/workflows/blockifier_post-merge.yml
@@ -5,7 +5,11 @@ on:
     types:
       - closed
     paths:
+      - '.github/workflows/blockifier_post-merge.yml'
       - 'crates/blockifier/**'
+      - 'crates/native_blockifier/**'
+      - 'scripts/requirements.txt'
+
 jobs:
   if_merged:
     if: github.event.pull_request.merged == true
@@ -29,4 +33,4 @@ jobs:
 
       - run: |
           pip install -r crates/blockifier/tests/requirements.txt
-          cargo test -- --include-ignored
+          cargo test -p blockifier -p native_blockifier -- --include-ignored