-
Notifications
You must be signed in to change notification settings - Fork 87
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
feat: integration tests #1070
feat: integration tests #1070
Conversation
fab9231
to
325b411
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @sash-a! It's really satisfying to see my chaotic shell script evolve into a more sophisticated and cleaner test version 😅💃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this @sash-a 🔥
Just some minor suggestions
Co-authored-by: Ruan de Kock <33461981+RuanJohn@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great! Thank you @sash-a 🔥
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @sash-a, I made a final optional request, If you believe it's unnecessary, feel free to ignore it 😅
stale, comments addressed and 2 approving reviews
What?
Adds integration tests
todo:
closes #1049