Marc Siegel

Results 122 comments of Marc Siegel

I made a test case for this (confirming the issue) using the following `test.rb`: ```ruby require "memo_wise" class A prepend MemoWise def answer 42 end memo_wise :answer end class B...

Confirmed that removing `memo_wise` in the parent class `A` fixes the problem. Seems like we are having a name collision between the super and sub method?

Any idea of a timeline for this? Am I the only person in the world who cannot install Leapp CLI until this is resolved?

Thanks and apologies - how do I try this? I’ve been installing on Mac OS using Homebrew?

@ericvilla Thank you so much! I can confirm that version v0.1.65 installed successfully using `brew install Noovolari/brew/leapp-cli` -- all set!

What if we change MemoWise to actually error out if we use include instead of prepend?

QQ: What would happen if everything stayed the same (that is: `include ModuleMethods` at the top, before the `def initialize`), but in `initialize` it calls `super` either at beginning or...

@JacobEvelyn I am wondering if we should consider making use of [Module#method_added](https://ruby-doc.org/3.2.2/Module.html#method-i-method_added) as a callback, and **if the method added is** `#initialize`, take some action?

One potential downside is that this code enforces that fields which have non-equal `#hash` values will cause the Value objects containing them to be unequal. What is the Ruby language...

Ok, interesting. One data point on "can unequal `hash` imply not `eql?`" A very common example of instances of different classes attempting full interchangeability with one another is ActiveSupport's [TimeWithZone](http://api.rubyonrails.org/classes/ActiveSupport/TimeWithZone.html),...