Skip to content

Commit

Permalink
➖ remove make as the automation tool
Browse files Browse the repository at this point in the history
This tool is not common in the Solidity ecosystem and it is not worth to
maintain it.
  • Loading branch information
qd-qd committed Nov 30, 2023
1 parent b73d19b commit fccadcd
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 164 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/quality-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ jobs:
run: npm ci

- name: Run the linter and the formatter in check mode
run: make quality
run: forge fmt --check && npx solhint "{script,src,test}/**/*.sol"

- name: Add lint summary
run: |
Expand All @@ -51,7 +51,7 @@ jobs:
uses: foundry-rs/foundry-toolchain@v1

- name: Compile the contracts and print their size
run: make compile-s
run: forge compile --sizes

- name: Add compile summary
run: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
uses: foundry-rs/foundry-toolchain@v1

- name: Build the documentation
run: make doc
run: forge doc && forge doc --build

- name: Deploy the documentation
uses: JamesIves/github-pages-deploy-action@v4
Expand Down
3 changes: 0 additions & 3 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,6 @@ broadcast/*/31337/
# coverage generated by genhtml
report

# makefile generated files
.make.*

# autogenerated documentation
docs

Expand Down
8 changes: 4 additions & 4 deletions lefthook.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@ pre-push:
commands:
lint:
tags: quality
run: make lint
run: forge fmt --check && npx solhint "{script,src,test}/**/*.sol"
format:
tags: quality
run: make format
run: npx prettier --check "**/*.{json,md,yml}"
test:
run: make test
run: forge test
build:
run: make compile
run: forge compile
154 changes: 0 additions & 154 deletions makefile

This file was deleted.

0 comments on commit fccadcd

Please sign in to comment.