Merge pull request #70 from lidofinance/feature/slither-fix

Resolve CI issues
lidofinance · May 7, 2024 · 6326f7a · 6326f7a
2 parents c5f7c5f + 3fd946b
commit 6326f7a
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 40 deletions.
diff --git a/.github/workflows/slither.yml b/.github/workflows/slither.yml
@@ -2,9 +2,10 @@ name: Slither Analysis
 
 on:
   push:
-    branches: [develop, master]
+    branches:    
+      - '*' 
   pull_request:
-    branches: [develop]
+    branches: [develop, master]
 
 jobs:
   analyze:
@@ -19,15 +20,18 @@ jobs:
       - uses: actions/setup-python@v4
         with:
           python-version: "3.10.6"
-
+
+      - name: Install Foundry
+        uses: foundry-rs/foundry-toolchain@v1
+
       - name: Install poetry requirements
         run: >
           curl -sSL https://install.python-poetry.org | python - &&
           poetry install
 
       - name: Run slither
         run: >
-          poetry run slither . --sarif results.sarif --fail-none
+          poetry run slither . --sarif results.sarif --fail-none --compile-force-framework foundry
 
       - name: Check results.sarif presence
         id: results

diff --git a/.github/workflows/verify-bytecode.yml b/.github/workflows/verify-bytecode.yml
diff --git a/remappings.txt b/remappings.txt
@@ -0,0 +1 @@
+OpenZeppelin/[email protected]/=dependencies/OpenZeppelin/[email protected]