-
Notifications
You must be signed in to change notification settings - Fork 299
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
Write tests #59
Labels
Comments
cpina
added
enhancement
New feature or request
help wanted
Extra attention is needed
labels
Jan 22, 2022
This was
linked to
pull requests
Jan 22, 2022
This was referenced Jan 22, 2022
Hey, If this issue is still up for grabs, I'd like to help out! |
@KaviiSuri (and anyone else) : apologies for missing to reply you in February! Yes, this is still for grab. I'm opened to proposals on how to test it (which parts to mock / fake, test the final result would be nice not only the git call, etc.). |
@cpina No issues, I'll research a bit and come up with a few options! |
Open
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
(Maintainer here)
Before adding more features and merging PRs (unless they fix an error) I want to write tests for the github action (should have been done long time ago).
Testing the action is getting harder and harder as we add features and I really want to avoid regressions.
I might have some time towards end of February 2022 but if anyone wants to have a look at writing tests it might speed up PR and issues.
I haven't planed yet how to implement the tests. If someone wants to help it might be a good idea to discuss options here.
Thanks for using it and for your patience!
The text was updated successfully, but these errors were encountered: