[Grok-dev] Re: SVN: grok/trunk/ merged ksmith_mcweekly-layers-011 branch, all tests pass

[Kevin Smith]

Fixed now, thanks!

Philipp von Weitershausen wrote:
> Kevin Smith wrote:
>> +
>> +class ILayerGrokker(martian.ClassGrokker):
>> +    component_class = grok.IGrokLayer
>> +
>
>
> This grokker doesn't do anything. Why is it there?
>
> Also, layers are interfaces. Interfaces aren't classes, even though 
> you happen to use the 'class' statement to create them. So even if 
> this grokker did anything, it wouldn't be called anyway, because 
> subinterfaces of IGrokLayer aren't classes and this grokker is a 
> ClassGrokker.
>
> Other than that, cool stuff. Thanks for getting this merged!
>
>