Skip to content

Conversation

@makivlach
Copy link

I've made this pull request in regards to this issue: #178.

I didn't have time to write tests, though. Please, someone, take this on.

@laurence-myers
Copy link
Member

Thanks for tackling the feature request!

We can't accept untested changes. Tests help document the expected/desired behaviour. Once you, or someone else, contributes tests, we can review and merge these changes.

The readme will also need to be updated, to document the feature.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants