Skip to content

Commit

Permalink
Merge pull request #68 from bgd-labs/feat/upgrade-tooling
Browse files Browse the repository at this point in the history
chore: upgrade tooling
  • Loading branch information
kyzia551 authored Oct 21, 2024
2 parents ec33f4f + a6f8d82 commit 18c4992
Show file tree
Hide file tree
Showing 11 changed files with 1,377 additions and 1,068 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/comment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:
comment:
name: Comment Bot
runs-on: ubuntu-latest
# workflow run triggeres on all types of "completed" including "cancelled" and similar
# workflow run triggers on all types of "completed" including "cancelled" and similar
# we want this action to only run on a success & failure though
if: ${{ github.event.workflow_run.conclusion == 'success' || github.event.workflow_run.conclusion == 'failure' }}
steps:
Expand Down
20 changes: 13 additions & 7 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,16 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@0a44ba7841725637a19e28fa30b79a866c81b0a6
with:
submodules: recursive
- uses: oven-sh/setup-bun@8f24390df009a496891208e5e36b8a1de1f45135
- name: Install node dependencies
run: bun install
node-version-file: .nvmrc
cache: "npm"

- name: install
run: npm ci --prefer-offline --no-audit

- name: lint
run: bun run lint --check
run: npm run lint

test:
name: Foundry build n test
Expand All @@ -33,15 +36,18 @@ jobs:
- uses: actions/checkout@v4
with:
submodules: recursive
- uses: oven-sh/setup-bun@8f24390df009a496891208e5e36b8a1de1f45135
- uses: actions/setup-node@0a44ba7841725637a19e28fa30b79a866c81b0a6
with:
node-version-file: .nvmrc
cache: "npm"

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@8f1998e9878d786675189ef566a2e4bf24869773
with:
version: nightly

- name: Install node dependencies
run: bun install
run: npm ci --prefer-offline --no-audit

- name: Run Forge build
run: |
Expand Down
4 changes: 2 additions & 2 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@ downloads/
.vscode
lcov*

# well, looks strange to ignore package-lock, but we have only pretter and it's temproray
package-lock.json
yarn.lock
bun.lockb
node_modules

# ignore foundry deploy artifacts
Expand Down
1 change: 1 addition & 0 deletions .nvmrc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
20
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,8 @@ cp .env.example .env

forge install

# optional, to install prettier
bun install
# required for tests & linting
npm install
```

<br>
Expand Down
Binary file removed bun.lockb
Binary file not shown.
5 changes: 1 addition & 4 deletions certora/stata/harness/pool/SymbolicLendingPool.sol
Original file line number Diff line number Diff line change
Expand Up @@ -93,10 +93,7 @@ contract SymbolicLendingPool {
return reserve.configuration;
}

function getVirtualUnderlyingBalance(
address asset
) external view virtual returns (uint128) {
function getVirtualUnderlyingBalance(address asset) external view virtual returns (uint128) {
return reserve.virtualUnderlyingBalance;
}

}
Loading

0 comments on commit 18c4992

Please sign in to comment.