Closed camilleryr closed 4 months ago
Hey @camilleryr, I would actually prefer if this was just the change as discussed in the issue - I actually find that cleaner and given it's a lower diff, it's easier to backtrack through later, too. Hope that's not a problem!
Hey @camilleryr, I would actually prefer if this was just the change as discussed in the issue - I actually find that cleaner and given it's a lower diff, it's easier to backtrack through later, too. Hope that's not a problem!
That works for me - I have updated the PR
Resolves #347
This is the same logic as mentioned in the issue, just cleaned up a bit - let me know if you would like to see any changes!