Skip to content

Commit

Permalink
chore: try to put explicit compilation in slither ci
Browse files Browse the repository at this point in the history
  • Loading branch information
0xtekgrinder committed Nov 3, 2023
1 parent 5a835e7 commit e60ee51
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 9 deletions.
14 changes: 5 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,6 @@ on:
branches:
- "main"

defaults:
run:
shell: bash

jobs:
lint:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -147,7 +143,7 @@ jobs:
version: nightly

- name: Run Foundry tests
run: yarn test:fuzz
run: npm run test:fuzz
env:
FOUNDRY_FUZZ_RUNS: "5000"

Expand Down Expand Up @@ -185,7 +181,6 @@ jobs:
echo "✅ Uploaded to Codecov" >> $GITHUB_STEP_SUMMARY
slither-analyze:
needs: ["build", "lint"]
runs-on: "ubuntu-latest"
permissions:
actions: "read"
Expand All @@ -202,9 +197,10 @@ jobs:
version: nightly

- name: Compile foundry
run: |
forge clean
forge build --build-info --skip */test/** */scripts/** --force
run: forge build --build-info --skip */test/** */scripts/** --force

- name: See output
run: ls -la out

- name: "Run Slither analysis"
uses: "crytic/[email protected]"
Expand Down
1 change: 1 addition & 0 deletions slither.config.json
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
{
"detectors_to_exclude": "naming-convention,reentrancy-events,solc-version,timestamp",
"filter_paths": "(lib|test|external)",
"solc_remaps": [
"ds-test/=lib/ds-test/src/",
Expand Down

0 comments on commit e60ee51

Please sign in to comment.