Skip to content

Conversation

@vegardmeh
Copy link
Contributor

See PR for terminal-mines: joelekstrom/terminal-mines#13

@joelekstrom
Copy link
Owner

Thanks for the contribution! It looks good to me and am happy to include this functionality but I'd like for a unit test to verify it (examples here) and C++ API support example.

It shouldn't be much work. If you need help with that let me know but then it might take longer before I get to it

@joelekstrom
Copy link
Owner

Looks great, thanks for adding tests!

@joelekstrom joelekstrom merged commit 82362c2 into joelekstrom:master Apr 17, 2025
1 check passed
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