m-delarosa / mod_0_skills

0 stars 0 forks source link

Assessment Results #1

Open damwhit opened 5 years ago

damwhit commented 5 years ago

Hey Mike, nice work on the assessment. You are officially technical ready. For the most part, great job with your naming conventions - it was really clear to me what each attribute or method did. Also, good job using the correct syntax on your data types.

One thing to keep in mind moving forward: isOn would likely be a better name than ovenOn since we know that it applies to an instance of an oven.

m-delarosa commented 5 years ago

Thanks @damwhit, I really appreciate the feedback. Using isOn does make more sense. Please let me know if there is anything else you think I can work on.