Closed jcapona closed 1 year ago
This pull request fixes 2 alerts when merging 5a0299e69feea61e0c84e6fcde957016b8b6c80e into de41307f005ebf24bf3ac59ebf2294ba783afe3f - view on LGTM.com
fixed alerts:
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.
Base: 60.38% // Head: 59.87% // Decreases project coverage by -0.51%
:warning:
Coverage data is based on head (
8d50295
) compared to base (c615000
). Patch coverage: 45.62% of modified lines in pull request are covered.:exclamation: Current head 8d50295 differs from pull request most recent head 447a183. Consider uploading reports for the commit 447a183 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.
Main changes
LightSensor
,SoundSensor
,Potentiometer
&UltrasonicSensor
) to simulation sprites.Screenshots (feature, test output, profiling, dev tools etc)
[insert screenshots here]
Other notes (e.g. implementation quirks, edge cases, questions / issues)
-
Manual testing tips
-
Tag anyone who definitely needs to review or help
-