search

Cyfrin / foundry-nft-cu

21 stars 22 forks source link

Event Log Fixed - MoodNft #8

Closed theirrationalone closed 1 year ago

theirrationalone commented 1 year ago
  1. Modified: /src/MoodNft.sol
  2. Updated: /test/MoodNftTtest.sol

Tests Passed without any issue 👇

Screenshot (139)

PatrickAlphaC commented 1 year ago

Thanks for this!

So a BIG thing you can see here is that the formatting changed a lot, making this PR difficult to review. Can you remove all the formatting changes you made?

PatrickAlphaC commented 1 year ago

ty!

theirrationalone commented 1 year ago

Can we use this syntax instead? I don't love the leading underscore

Why not. Ser! ;`)