Refactoring - Githubissues

Cyfrin / foundry-smart-contract-lottery-cu

47 stars 38 forks source link

Refactoring #6

Closed ShayanShamsi closed 1 year ago

ShayanShamsi commented 1 year ago

Corrected a couple of wrong comments.
Made use of the enteredRaffle modifier in unit tests for cleaner code and to reduce repetition.
Rearranged ordering of functions in script/ and src/ directories to follow the Solidity style guide.

PatrickAlphaC commented 1 year ago

Closing, a lot of reformatting here. Happy to re-open if the formatting is fixed. When you make a PR but the formatting is different, it's really hard to review!

Nice work going above and beyond on these though!