Closed jspaezp closed 6 months ago
I am sincerely sorry for the lack of context here! Your comment is overly nice. This PR was not meant to be reviewed to be merged but rather to be pointed to in the issue #153. I apologize for the misunderstanding!
Thanks for sending this! Unfortunately this is currently very hard to review as you mix your own change with the already merged PRs #184 and #186. Can you please rebase your change on the top of the current main branch?