cool


On Wed, Jan 8, 2014 at 6:16 PM, imranr <notifications@github.com> wrote:

> In
> core/src/main/scala/com/quantifind/sumac/validation/AnnotationValidationFunctions.scala:
>
> > @@ -46,4 +47,17 @@ object RangeCheck extends Function4[Any, Any, Annotation, String, Unit] {
> >        case _ => ()
> >      }
> >    }
> > +}
> > +
> > +object FileExistsCheck extends Function4[Any, Any, Annotation, String, Unit] {
> > +
> > +  def apply(defaultValue: Any, currentValue: Any, annot: Annotation, name: String) {
> > +    Option(currentValue).map(_.toString) match {
>
> karthik and I discussed this in chat. Personally, I like the behavior as
> is. If there is only a default value, and that value exists, I think it
> should be valid. I think @Required is really kind of a special beast that
> should not be emulated.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/quantifind/Sumac/pull/31/files#r8737446>
> .
>