threshold-network / solidity-contracts

39 stars 18 forks source link

Cleaning up Slither and Linting - unblocking CI checks #144

Closed dimpar closed 1 year ago

dimpar commented 1 year ago

This PR cleans up existing code by linting a test file and disables "incorrect-equality" slither check for one of the require statements.

These changes are needed to pass CI checks in other PR and currently is a blocker. For some reason, CI didn't catch linting and slither issues when the actual code was merged to the main branch here https://github.com/threshold-network/solidity-contracts/commit/f468a7d8a3aa84bf65f0819b521302efe46d016b