Skip to content

Commit

Permalink
Merge pull request #118 from Bananapus/fix/ci
Browse files Browse the repository at this point in the history
fix: ci npm and slither issues
  • Loading branch information
xBA5ED authored Mar 23, 2024
2 parents 8ba615c + 43a665b commit 04cbd09
Show file tree
Hide file tree
Showing 5 changed files with 973 additions and 243 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/slither.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ on:
jobs:
analyze:
runs-on: ubuntu-latest
env:
NODE_ENV: production
steps:
- uses: actions/checkout@v4
with:
Expand All @@ -19,7 +21,7 @@ jobs:
with:
node-version: latest
- name: Install npm dependencies
run: npm ci
run: npm ci --production
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
- name: Run slither
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,12 @@ jobs:
with:
node-version: latest
- name: Install npm dependencies
run: npm ci
run: npm ci --production
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
- name: Run local tests
run: forge snapshot --fail-fast --summary --detailed
- name: Run fork tests
run: FOUNDRY_PROFILE=fork forge snapshot --fail-fast --summary --detailed
- name: Check contract sizes
run: forge build --sizes
run: forge build --sizes --skip SphinxUtils
6 changes: 4 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ curl -L https://foundry.paradigm.xyz | sh
You can download and install dependencies with:

```bash
npm install && forge install
npm install --production && forge install
```

If you run into trouble with `forge install`, try using `git submodule update --init --recursive` to ensure that nested submodules have been properly initialized.
Expand All @@ -52,7 +52,9 @@ To learn more, visit the [Foundry Book](https://book.getfoundry.sh/) docs.

## Scripts

For convenience, several utility commands are available in `package.json`.
In order to be able to run deployment scripts you have to install the npm `devDependencies`, to do this run `npm install`.

For convenience, several utility commands are available in `package.json`.

| Command | Description |
| --------------------------------- | -------------------------------------- |
Expand Down
Loading

0 comments on commit 04cbd09

Please sign in to comment.