glomstea / MI-449-SS17-741-js-collect-and-display-information-kPt9Fl

0 stars 0 forks source link

Project Feedback #1

Open glomstea opened 7 years ago

glomstea commented 7 years ago

@KatieMFritz Can you take a look at this? It's hosted here and meets the following criteria:

KatieMFritz commented 7 years ago

Hey Grant, nice job! 🔥 💥 ✨ 🍪

:shipit:

The only thing I would recommend you do differently in the future is get rid of your test files before submitting/publishing. It's a great strategy to save backup or alternate versions of code while you're working, but in production (the published project), they can confuse other developers, confuse your future self who forgets what they are for, and in some cases even hurt the performance of the site or app. Does that make sense? 🙂