-
Notifications
You must be signed in to change notification settings - Fork 794
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add a pipeline and some unit tests ? #34
Labels
enhancement
New feature or request
Comments
Mocha would be great :) |
+1 Are you aiming for 80% global coverage? @navopw @wassef911 |
Would be nice but this project is currently in rapid development phase. Will start with the tests in a few days :D |
There already was a PR for tests but I had to decline because the methods changed. Will merge this sooner or later tho. |
Makes sense!
…On Sun, Feb 19, 2023, 4:18 PM Navo ***@***.***> wrote:
There already was a PR for tests but I had to decline because the methods
changed.
#49 <#49>
Will merge this sooner or later tho.
—
Reply to this email directly, view it on GitHub
<#34 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOFMTPWRSWTCTRIU347ZPLLWYKL4HANCNFSM6AAAAAAVALKP6Q>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
understood |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
any suggestions on what to use? Mocha? Tape?
The text was updated successfully, but these errors were encountered: