dkubb / axiom

Simplifies querying of structured data using relational algebra
https://github.com/dkubb/axiom
MIT License
459 stars 22 forks source link

Adamantium zero arity #19

Closed mbj closed 11 years ago

mbj commented 11 years ago

This PR fixes a case where veritas is memoizing a method that has a nonzero arity.

It includes two commits where only one should end up in master.

As there is no git source for reek that contains the Irresponsible module fix and is compatible to heckle ci is subjected to fail. The reek issue needs to be addressed as currently reek ^ heckle .

dkubb commented 11 years ago

It looks like Enumerable#to_set does actually take arguments, and since I don't really like changing the interface of overridden instance methods I may just avoid memoizing it altogether.