Ilya Grigorik
Ilya Grigorik
We should update the code to use the provided crc32 implementation.. no reason to duplicate it.
> Added tests to check that both old and new style can be loaded Hmm, do we actually want that? Generally, I'm not a fan of carrying this kind of...
@marcheiligers it's not performance impact, as much as unnecessary anchors / code moving forward. With that said, I guess it's not too big of a hurdle here.
Hmm, I think we're well past due on a release... Can you try from master?
Give it a shot! :)
Yep, makes sense! In terms of plumbing, I guess you'd need to filter the git history for files that are only in that directory..
Hmm, do you have some example code for this? Admittedly, not all that familiar with the grit API yet..
Good question. At the time I had a demo project I was testing this on =|
Is this necessary? We already declare it in the gemspec: https://github.com/igrigorik/decisiontree/blob/master/decisiontree.gemspec#L13
Well, in either case, that's not much of a decision tree.. a) same attributes resolving to different labels - current code just picks the last value, so all of the...