-
Notifications
You must be signed in to change notification settings - Fork 11
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 unit test workflow #266
Conversation
Test coverage of modified packages: |
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.
Please describe the workflow in the readme file. Maybe add some nice workflow status badge?
https://docs.github.com/en/actions/monitoring-and-troubleshooting-workflows/adding-a-workflow-status-badge
Regarding both comments I'd wait with the readme update and badge until the tests are fully functional and pass to avoid always having a red badge. As for the information that tests are run on hardware I'd see that info somewhere near/on the badge in readme. I'm not sure about the proposed filename since we don't need to distinguish between "hardware" and "non-hardware"(GitHub-hosted). What do you think, is it a good option in your opinion? |
Changes
panteher_utils
package ✅❗ SQUASH COMMITS! ❗