odyslam/foundry-upgrades

Feedback

odyslam opened this issue · 4 comments

odyslam commented
  • use types.sol to import the proxyType enum to both deploy and proxytester
  • deploy/set type in one action
  • more intuitive API
  • It should also verify the storage root to ensure no other slots have been manipulated.
PatStiles commented

Hi! Willing to take on adding verification of the storage root. Let me know!

odyslam commented

@PatStiles Please do !

I have started playing with the package again and have identified a couple of API improvements.

alexbakers commented

Please add more documentation.

alexbakers commented

This documentation is very clear:
https://github.com/OpenZeppelin/openzeppelin-upgrades/tree/master/packages/plugin-hardhat/

Could you make similar documentation for foundry.

  • 👍6