qassisju / MI-449-SS17-741-html-more-kinds-of-content-OSB7Pe

0 stars 0 forks source link

Project Feedback #1

Open qassisju opened 7 years ago

qassisju commented 7 years ago

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

stuartpearman commented 7 years ago

Hey justin, the link didn't seem to work, did you push to Github pages?

qassisju commented 7 years ago

Hey stuart, sorry, I must have forgotten. It should be up now

stuartpearman commented 7 years ago

"Sometimes if you find baby kangaroos you get to keep them" 😆

This looks good Justin! The only thing I would change is your indentation. In this class we're using 2-space indentation. This is just the standard for this class. As you probably know, indentation is a heated topic in the programming world 🔥 . The Writing code for humans to read slide from a previous lesson talks about how to set your default indentation in atom.

qassisju commented 7 years ago

Hi Stuart,

I changed the spacing, now it should be better

stuartpearman commented 7 years ago

Yep, looks good! :shipit: