slither icon indicating copy to clipboard operation
slither copied to clipboard

0.8.4 release

Open 0xalpharush opened this issue 2 years ago • 2 comments

Issues:

  • [x] https://github.com/crytic/slither/issues/1001
  • [ ] https://github.com/crytic/slither/issues/1174
  • [x] https://github.com/crytic/slither/issues/1173
  • [x] https://github.com/crytic/slither/issues/1211
  • [x] https://github.com/crytic/slither/pull/1227
  • [ ] https://github.com/crytic/slither/issues/1019

Nice to have:

  • [ ] https://github.com/crytic/slither/issues/511
  • [x] https://github.com/crytic/slither/issues/482
  • [ ] https://github.com/crytic/slither/issues/922
  • [x] https://github.com/crytic/slither/issues/1091

PRs to review:

Parse top level enums:

  • [x] https://github.com/crytic/slither/pull/1245

Fix ternary parsing of unused tuple type:

  • [ ] https://github.com/crytic/slither/pull/1217

Fix call conversion when library function and state variable conflict:

  • [x] https://github.com/crytic/slither/pull/1230

Update dependencies, CI, move to python 3.8:

  • [x] https://github.com/crytic/slither/pull/1178

Support top level constants in Yul:

  • [x] https://github.com/crytic/slither/pull/1188

Filter calldata optimization detector:

  • [x] https://github.com/crytic/slither/pull/1022

Document questions/ helpful info:

  • [x] https://github.com/crytic/slither/pull/1047

PR ready to merge:

https://github.com/crytic/slither/pulls?q=is%3Apr+is%3Aopen+review%3Aapproved

0xalpharush avatar Jun 22 '22 01:06 0xalpharush

I think we should also consider adding mypy for a subset of the project, at the very least. Namely, the following directories: core/, detectors/, slithir/. There were a couple bugs recently that would have been caught by type checking.

0xalpharush avatar Jul 20 '22 21:07 0xalpharush

Another thing to do for the release: update the solc version detector, and document our process

montyly avatar Aug 11 '22 12:08 montyly