Closed angusjfw closed 1 year ago
Base: 56.97% // Head: 58.72% // Increases project coverage by +1.74%
:tada:
Coverage data is based on head (
b243428
) compared to base (b8e144f
). Patch coverage: 80.84% of modified lines in pull request are covered.:exclamation: Current head b243428 differs from pull request most recent head 3bc099f. Consider uploading reports for the commit 3bc099f to get more accurate results
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
This pull request introduces 6 alerts when merging e6478fd4a44158fad4de58adb6096cf38db7351b into fc2f43a4d2fa08715e2431987bc2a5890c455d4c - view on LGTM.com
new alerts:
This pull request introduces 6 alerts when merging d86e69b03f9704d487701ea97e96bc4ff3d1ae3f into fc2f43a4d2fa08715e2431987bc2a5890c455d4c - view on LGTM.com
new alerts:
This pull request introduces 6 alerts when merging 48d1c6db7a3c6a62587eb85717f9f523324af826 into fc2f43a4d2fa08715e2431987bc2a5890c455d4c - view on LGTM.com
new alerts:
This pull request introduces 8 alerts when merging ff4d85b8ceac1f129e4f80f4a8b1728f92677e88 into fc2f43a4d2fa08715e2431987bc2a5890c455d4c - view on LGTM.com
new alerts:
This pull request introduces 8 alerts when merging b24342889cabfb8cccde447197fda71159e8092f into b8e144ffcd514c664f67515e0707f8d0eb7cc68e - view on LGTM.com
new alerts:
This pull request introduces 8 alerts when merging 236bdd8f26d569bfa3fdc780cba075c282a44a44 into b8e144ffcd514c664f67515e0707f8d0eb7cc68e - view on LGTM.com
new alerts:
This pull request introduces 2 alerts when merging b404fb8d93599a042354d8e76b1c4d8e587d9048 into b8e144ffcd514c664f67515e0707f8d0eb7cc68e - view on LGTM.com
new alerts:
This pull request introduces 2 alerts when merging 3bc099f89e4dd7df07eb03949d6b2a3ec9ee65cf into b8e144ffcd514c664f67515e0707f8d0eb7cc68e - view on LGTM.com
new alerts:
Main changes
-
Screenshots (feature, test output, profiling, dev tools etc)
[insert screenshots here]
Other notes (e.g. implementation quirks, edge cases, questions / issues)
--dist no
as well as-n0
. Loadfile ensures tests in the same file are not run concurrently, which helps to avoid an issue with tests using gpiozero, where closing a component on a particular port in one process causes it to be closed in others.Manual testing tips
-
Tag anyone who definitely needs to review or help
-