Mockery panics when configuration contains a package with a null value and recursive is true about mockery HOT 4 CLOSED

@LandonTClipp I wanted to add that I can to help work on this if you think it is an issue worth addressing. I have found a huge amount of value from this project and would be happy to contribute to it if it would add value.

from mockery.

Damn was hoping no one would notice that 😂

I'm glad to hear you're enjoying the project so much. There is a lot more interesting stuff coming down the pipe. Yes please do send in PRs, although let's see if we can narrow down why this happens.

The config merging code is quite complex but I didn't have a good tool at my disposal that makes this easier. Let's see if we can find where this is happening.

The other thing... showconfig is kind of broken due to a bug noted in another issue (I'm on mobile so can't easily find it now). We should figure out how to fix that because it sometimes hangs indefinitely.

from mockery.

FYI in the linked PR, I fixed the showconfig command and also the issue mentioned above with particular kinds of config causing mockery to crash. Please take a look @dlm when you get the chance, thanks for reporting!

from mockery.

@LandonTClipp sorry for the slow reply. (Hectic Nov & Dec). I just did the upgrade and it works perfectly!! Thank you very much for getting this resolved!!

from mockery.