alphagov/panopticon — comment on commit 2c0fe3bf8be42ecc73e9e89939cc4cbac085e85c
File: lib/contact_listener.rb line 42
By craigw:

Oops, I didn't realise I had left this in the original example. I meant to change it to this:

initialize_with :logger
default_value_of :logger, NullLogger.instance

That way someone somewhere else worries about logger implementation and configuration, our objects don't need to care.

No panic now - don't let me distract you - although I might change all the instances of this tonight if no one yells... Does setting a NullLogger as a default seem like a reasonable approach?