search

ga-wdi-boston / mongoose

An introduction to mongoose
Other
5 stars 136 forks source link

Remove `if(true)` #35

Open payne-chris-r opened 8 years ago

payne-chris-r commented 8 years ago

Why have the true at all? This is the same as writing if(true)

if (true || givenName) {
        create(givenName, surname, dob, gender, height, weight);
      }
gaand commented 8 years ago

Who wrote the code, @payne-chris-r :question:

Also, this is the same as not having the if at all :running_man: :hammer: :robot:

payne-chris-r commented 7 years ago

Looks like Matt.

Reorganize code files for code-along
GA-MEB authored on Mar 8
102       if (true || givenName) {

I think this should be rewritten as 

if (givenName) {
        create(givenName, surname, dob, gender, height, weight);
      }
gaand commented 7 years ago

I'd need more context to know what makes sense.