SunWeb3Sec/DeFiHackLabs

Refactoring proposal

Closed this issue · 3 comments

Hello,

I am starting to learn this codebase and I believe it could benefit from a structural refactor.

Currently, the README contains descriptions from various exploits, in chronological order, together with foundry reproducers. The issue is that the reproducers are all on the src folder, without any ordering whatsoever.

I propose a slight change on that structure:

images
  |_ ...
academy
  |_ ...
hacks
  |_ 2024
    |_ 01
      |_ OrbitChain_exp.sol

If you agree on this refactor, I can submit a PR

Hey, it makes sense to me. However, we can wait one day in case anyone has objections.
Thanks a lot!

I guess I'll wait for the approval of #547, otherwise I'll make their lives harder

@aviggiano I have closed #547 , you can go ahead. thanks a lot.