-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[sensorcommunity] State pattern for dB, Percent and Microgram/m3 added #15692
Conversation
Signed-off-by: Bernd Weymann <bernd.weymann@gmail.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.
Thanks! Only one minor remark.
...g.sensorcommunity/src/test/java/org/openhab/binding/sensorcommunity/internal/NumberTest.java
Outdated
Show resolved
Hide resolved
bundles/org.openhab.binding.sensorcommunity/src/main/resources/OH-INF/thing/thing-types.xml
Show resolved
Hide resolved
So for me this PR is OK regarding the changes on state pattern. I did not look at tests as @jlaur already commented. |
@jlaur : I let you the lead on the merge of this PR. |
@weymann : can you please look at review comments so that your PR can be merged please. |
Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>
fdfda11
to
acda68d
Compare
Done |
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.
LGTM, thank you
openhab#15692) * State pattern for dB, Percent and Microgram/m3 added --------- Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>
openhab#15692) * State pattern for dB, Percent and Microgram/m3 added --------- Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com> Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
State pattern for dB, Percent and Microgram/m3 added