Conversation
|
@pappas999 Thank you for taking the time to create this PR. I see that you updated all the |
|
thanks @andreipope , I updated the previous lessons as well. Everything should be consistent now |
TomiOhl
left a comment
TomiOhl
left a comment
There was a problem hiding this comment.
Nice work. I have some thoughts on the styling, but nothing major.
|
thanks @TomiOhl , I incorporated the feedback given. Please take a look when you can, thanks! |
TomiOhl
left a comment
TomiOhl
left a comment
There was a problem hiding this comment.
Nice work! Looks fine to me now. Except that I found this one more thing, let's fix up that one too and it'll be ready to go.
en/19/11.md
Outdated
| zombies.push(Zombie(_name, _dna)); | ||
| } | ||
|
|
||
| // Delete the function below |
There was a problem hiding this comment.
To be complete, a "1." should be added here
|
thanks @TomiOhl , i added the change, as well as a couple others I spotted on other pages while doing a check through again. |
TomiOhl
left a comment
TomiOhl
left a comment
There was a problem hiding this comment.
Nice! Didn't notice them since there wasn't even a blank space after the slashes.
From my side, I think it's ready to merge!
3 participants
updated Chainlink VRF section to use the new Chainlink VRF V2