Closed oliverlockwood closed 8 years ago
@kevinpotgieter FYI - we failed here :jack_o_lantern:
@balooo @jsravn please review ASAP
We try to review everything ASAP :smile:
@balooo well yes :stuck_out_tongue:
Ok merge it ASAP
I merged it ASAP
You guys :guitar:
Review fail by me in #24 :(
In #24 we discussed at length setting:
Unfortunately, in the actual implementation we got this the wrong way round, and our tests tested themselves rather than the production code.
This PR aims to fix that. I changed the tests so they started failing, and then changed the prod code so the tests went green again.
After this goes in we will need to issue another release.