Skip to content

Conversation

jaimedelano
Copy link

No description provided.

Copy link
Member

@ncw ncw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks very useful - thank you.

Can you combine this with #11 and make a single pull request please.

It would be useful to have a test for this - can you think of how to add one?

@ncw
Copy link
Member

ncw commented Sep 9, 2025

How are you doing with this @jaimedelano ?

@jaimedelano
Copy link
Author

I don't know how to do that.

@ncw
Copy link
Member

ncw commented Sep 13, 2025

I've done that for you in this PR

What do you think about adding tests?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants