Skip to content

Commit

Permalink
Merge pull request #27 from ethereum-optimism/tip/rvsol-lint
Browse files Browse the repository at this point in the history
rvsol: Lint CI job
  • Loading branch information
ImTei authored Mar 26, 2024
2 parents c72af6a + b479d58 commit 122e520
Show file tree
Hide file tree
Showing 6 changed files with 717 additions and 397 deletions.
11 changes: 11 additions & 0 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,17 @@ jobs:
- name: Run foundry tests
run: forge test -vvv --ffi
working-directory: rvsol
rvsol-lint:
runs-on: ubuntu-latest
timeout-minutes: 20
steps:
- name: Checkout repository
uses: actions/checkout@v4
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
- name: Run lint
run: make lint-check
working-directory: rvsol

# go-lint:
# runs-on: ubuntu-latest
Expand Down
8 changes: 8 additions & 0 deletions rvsol/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,3 +9,11 @@ clean:
test:
forge test -vvv --ffi
.PHONY: test

lint-fix:
forge fmt
.PHONY: lint-fix

lint-check:
forge fmt && git diff --exit-code
.PHONY: lint-check
6 changes: 6 additions & 0 deletions rvsol/foundry.toml
Original file line number Diff line number Diff line change
Expand Up @@ -30,4 +30,10 @@ fs_permissions = [
{ access='read', path='./' }
]

[fmt]
line_length=120
multiline_func_header='all'
bracket_spacing=true
wrap_comments=true

# See more config options https://github.com/foundry-rs/foundry/tree/master/config
Loading

0 comments on commit 122e520

Please sign in to comment.