zeta-chain / protocol-contracts

Protocol contracts implementing the core logic of the protocol, deployed on ZetaChain and on connected chains
MIT License
70 stars 58 forks source link

Integrating Slither #165

Closed lumtis closed 4 months ago

lumtis commented 5 months ago

Adding Slither: https://github.com/crytic/slither

Adding in the CI as well.

The tool is already installed in app contracts: https://github.com/zeta-chain/app-contracts

fadeev commented 5 months ago

@Kali-Decoder please, let me know if I should assign this issue to you.

lumtis commented 5 months ago

Right now it can be run manually but the steps are a little bit annoying: On mac

brew install solc-select
brew unlink solidity
brew link --overwrite solc-select
solc-select install 0.8.7
solc-select use 0.8.7
slither contracts/...--solc-args "--allow-paths .,./node_modules" --solc-remaps @openzeppelin=node_modules/@openzeppelin

The config for running slither could be optimized as well. End goal would to have a script doing everything

Kali-Decoder commented 5 months ago

Yes you can assign me @fadeev

fadeev commented 5 months ago

@Kali-Decoder

Kali-Decoder commented 4 months ago

To run the commands mentioned, I'm utilizing a shell script. I can then add the script's run command to the package.json and incorporate it into the PR workflow.

fadeev commented 4 months ago

@Kali-Decoder please, update on progress.

fadeev commented 4 months ago

@Kali-Decoder please, update on progress or let me know if I can re-assign this issue. Thank you.

fadeev commented 4 months ago

@Kali-Decoder the deadline for this issue is Sunday evening UTC. If you're blocked on something, please, share it here.

Kali-Decoder commented 4 months ago

okay @fadeev