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
fix warnings #1867
fix warnings #1867
Conversation
Hi @TESTPERSONAL, It seems you haven't yet signed a CLA. Please do so here. Once you do that we will be able to review and accept this pull request. Thanks! |
mapOf() | ||
) | ||
} | ||
if (event != null && event.sensor.type == Sensor.TYPE_LIGHT) { |
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.
Think this is even better?
if (event != null && event.sensor.type == Sensor.TYPE_LIGHT) { | |
if (event?.sensor.type == Sensor.TYPE_LIGHT) { | |
mapOf() | ||
) | ||
} | ||
if (event != null && event.sensor.type == Sensor.TYPE_PROXIMITY) { |
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.
if (event != null && event.sensor.type == Sensor.TYPE_PROXIMITY) { | |
if (event?.sensor.type == Sensor.TYPE_PROXIMITY) { | |
if (lastUpdateSensor != null) { | ||
if (lastUpdateSensor.enabled) | ||
LastUpdateManager().sendLastUpdate(appContext, TAG) | ||
if (lastUpdateSensor != null && lastUpdateSensor.enabled) { |
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.
if (lastUpdateSensor != null && lastUpdateSensor.enabled) { | |
if (lastUpdateSensor?.enabled == true) { |
mapOf() | ||
) | ||
} | ||
if (event != null && event.sensor.type == Sensor.TYPE_STEP_COUNTER) { |
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.
if (event != null && event.sensor.type == Sensor.TYPE_STEP_COUNTER) { | |
if (event?.sensor.type == Sensor.TYPE_STEP_COUNTER) { | |
Hi @TESTPERSONAL, It seems you haven't yet signed a CLA. Please do so here. Once you do that we will be able to review and accept this pull request. Thanks! |
@nick2525 we really need to figure out your commit email issues. |
Summary
Just fix some warnings
Screenshots
Link to pull request in Documentation repository
Documentation: home-assistant/companion.home-assistant#
Any other notes