search

bjedrzejewski / chessmates

Statistics for your Lichess team
1 stars 3 forks source link

Fixes silly mistake that disabled logging #19

Closed janakerman closed 7 years ago

janakerman commented 7 years ago

When I converted these to static properties I failled to notice that the this context no longer made sense.

janakerman commented 7 years ago

@bjedrzejewski - Merging this myself. I think it's coming to the point now where I need to go back and write some unit tests to cover the current behaviour. How do you also feel about unit tests that confirm logging - are they necessary?