Skip to content

Conversation

@TechnikTil
Copy link

i know that theres no unit tests or whatev

it works tho

@Davvex87
Copy link
Owner

I'll work on implementing a proper unit test runner class for it soon.
Keeping this PR open in the meantime...

@TechnikTil
Copy link
Author

alr!

@TechnikTil
Copy link
Author

TechnikTil commented Apr 19, 2025

I'll work on implementing a proper unit test runner class for it soon. Keeping this PR open in the meantime...

i could also make a pr for proper unit testing if u dont wanna do that

@Davvex87 Davvex87 added the enhancement New feature or request label Apr 22, 2025
@Davvex87
Copy link
Owner

I have created a new PR which I will be pushing to the unit testing code, you can find it here: #4
I re-estate that I will keep this PR open. I will be merging it once I am done with the unit tests.

Feel free to suggest additions to the test class over on that PR, though I'm not sure if github will allow you to push changes to that one, if not then you can either paste your code in the PR comments for me to merge or create an entirely new PR and I'll soon deal with the conflicts ;)

@Davvex87
Copy link
Owner

I probably should've just created a new branch for this, I was in a rush. Whoops

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants