CodingTrain / Bizarro-Devin

12 stars 4 forks source link

create index.html and sketch.js #20

Closed JuggernautJf closed 5 months ago

JuggernautJf commented 5 months ago

added index.html and sketch.js using setupFiles command fixes #1

supercrafter100 commented 5 months ago

Hey there!

Please see the review comments on your other PR, they don't appear to be addressed here. (#16) I'm pretty much just talking about starting the classes with a capital letter (which isn't done here)

JuggernautJf commented 5 months ago

https://github.com/CodingTrain/Bizarro-Devin/pull/16#pullrequestreview-1965153067 << this one? ohhhhhhhh... ya doing it!!!😊

JuggernautJf commented 5 months ago

@supercrafter100 thanks!

JuggernautJf commented 5 months ago

@shiffman think its good to go then? 👉👈

shiffman commented 5 months ago

Wonderful, thank you @JuggernautJf I'm going to merge! I'll be working on this project a lot of over the weekend so I would suggest not making too many significant changes without opening an issue for discussion first, thank you for your generous spirit!