search

metricfu / metric_fu

A fist full of code metrics
http://metricfu.github.com/metric_fu
MIT License
624 stars 96 forks source link

Require digest #318

Closed bronzdoc closed 4 years ago

bronzdoc commented 4 years ago

Require digest so specs doen't fail on windows with: 

Error: NameError; message uninitialized constant MetricFu::Templates::MetricsTemplate::Digest.
bronzdoc commented 4 years ago

cc @etagwerker

etagwerker commented 4 years ago

@bronzdoc I'm curious, why didn't you put it at the top of the file?

bronzdoc commented 4 years ago

@etagwerker Since the only place where we use Digest in the file is in that method, it makes sense to me to require it there on demand just when we need it.