Skip to content

Conversation

dns2utf8
Copy link
Contributor

No description provided.

@dbrgn
Copy link
Contributor

dbrgn commented Jan 24, 2017

Maybe we should keep a CHANGELOG file that would need to be updated with every change?

@dns2utf8
Copy link
Contributor Author

Good point, will add one.

@dbrgn
Copy link
Contributor

dbrgn commented Jan 29, 2017

#9 is merged, feel free to rebase.

@dns2utf8 dns2utf8 force-pushed the hpa-rename branch 3 times, most recently from 1b66383 to 646fd4a Compare January 30, 2017 12:23
@dns2utf8 dns2utf8 requested review from rorist, dbrgn and rnestler January 30, 2017 12:24
@dns2utf8
Copy link
Contributor Author

Added and rebased

@dns2utf8 dns2utf8 added this to the API v0.14 milestone Jan 30, 2017
CHANGELOG.md Outdated
@@ -9,4 +9,4 @@ Changes should start with one of the following tags:

## v0.14 (unreleased)

- ...
- [changed] The unit `hPA` was renamed to `hPa` to match the SI unit
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd mention where this unit is found:

The unit hPA in sensors.barometer.unit was renamed to hPa to match the SI unit

@rnestler
Copy link
Member

@dns2utf8 ping

This should probably be merged after #9
Copy link
Member

@rnestler rnestler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! :shipit:

@dbrgn dbrgn merged commit ed00121 into master Oct 7, 2017
@dbrgn dbrgn deleted the hpa-rename branch October 7, 2017 11:02
dbrgn pushed a commit that referenced this pull request Jan 6, 2018
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.

3 participants