Approved.

fixed

1. Actual & Expected are flipped in the random test.

   test.assert_approx_equals

   test.assert_approx_equals(my_celsius_to_romer(1.0*temp), celsius_to_romer(1.0*temp))
   #                         ^^^^^^^^^^^^^^^^^^^            ^^^^^^^^^^^^^^^^
   

2. Please use random values for the random tests.

he probably copy pasted the content of this fork and did an edit himself xD

\o/

Rejected, since issue already resolved.

Diff --> this line added to the description.

Rejected, merge conflict & author changed the description without approving this fork.

_{Have 2 consolation upvotes for the effort instead. 🙂}

Approved

Fixed in your fork.

Approved.

1. overlooked that, fixed;
2. Replaced as well, so everything should be good now

Nice! Two more things.

1. Actual & expected are swapped in the random test

           expect    = new Person(personGuy).greet(whoGuy)
           actual    = "Hello #{whoGuy}, my name is #{personGuy}"
   #       ^^^^^^
   

2. This is not test breaking, but you can simplify the random test by using _.sample instead (Lodash ref)

           personGuy = persons[genIdx(persons.length-1)]
           whoGuy    = who[genIdx(who.length-1)]
           #           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
           #           can be replaced with _.sample(who), for example
   

alright it's reworked now

Already approved by someone

oh, I thought Coffeescript doesn't support mocha + chai

will rework rn, thx :3