Skip to content
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 specific gravity device class and units #32

Merged
merged 1 commit into from
Aug 30, 2022

Conversation

apt-itude
Copy link
Contributor

This is necessary for the Tilt BLE sensor here. Still have an open question about the units since this sensor is truly unitless. I made it just "SG" for now in case we need to just fill in something.

@codecov
Copy link

codecov bot commented Aug 30, 2022

Codecov Report

Merging #32 (8132a2f) into main (35c47b2) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   93.44%   93.47%   +0.03%     
==========================================
  Files          12       12              
  Lines         412      414       +2     
  Branches       25       25              
==========================================
+ Hits          385      387       +2     
  Misses         26       26              
  Partials        1        1              
Impacted Files Coverage Δ
src/sensor_state_data/__init__.py 100.00% <100.00%> (ø)
src/sensor_state_data/sensor/device_class.py 100.00% <100.00%> (ø)
src/sensor_state_data/units.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@bdraco
Copy link
Member

bdraco commented Aug 30, 2022

If you rebase and prefix the commit message with

feat:

And make the first character lowercase the CI should pass

@bdraco bdraco merged commit 59e4d76 into Bluetooth-Devices:main Aug 30, 2022
@bdraco bdraco mentioned this pull request Aug 30, 2022
@apt-itude apt-itude deleted the specific-gravity branch August 30, 2022 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants