foundry-upgrades icon indicating copy to clipboard operation
foundry-upgrades copied to clipboard

Feedback

Open odyslam opened this issue 2 years ago • 4 comments

  • 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.

odyslam avatar Mar 22 '22 20:03 odyslam

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

PatStiles avatar Jul 31 '22 00:07 PatStiles

@PatStiles Please do !

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

odyslam avatar Aug 04 '22 22:08 odyslam

Please add more documentation.

alexbakers avatar Aug 18 '22 01:08 alexbakers

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

Could you make similar documentation for foundry.

alexbakers avatar Aug 18 '22 01:08 alexbakers