Closed joemsak closed 2 years ago
@parndt I missed some necessary changes, this should resolve that.
Dearest @parndt what is the likelihood of this getting merged and then a release? 🙏This PR satisfies the documentation. Let me know if you have feedback. Thanks!
@joemsak I have been thinking about this change on and off all week! I've added a suggestion, let me know what you think
I also spotted a duplicate raise
which I've removed
Thank you!
Hey @parndt any chance on this making it to a release? Let me know your thoughts, thanks!
@joemsak yes, definitely, I will get to it as soon as I can. 😄
Version 5.5.0 pushed @joemsak
Turns out, the documentation I added in #995 wasn't 100% accurate. I had to add tests for such examples and improve the internal code to support the option. This should now provide the fully expected behavior.