-
Notifications
You must be signed in to change notification settings - Fork 190
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
Commit hash always 'unknown' #146
Comments
I also have the same 'error' |
Looking at merging this in for version 1.5.0 |
It still shows
|
@ciotlosm yeah, that's expected. Here's the relevant bit in the edge version Dockerfile
I didn't include this in stable version because it's using a known release of zigbee2mqtt by definition, which I figured would be the most relevant info. With that said, that release obviously is tied to a commit, and I guess it couldn't hurt to provide the hash as well, even if it is kind of redundant. |
So, will it finally be included? (I think having it printed is useful, although redundant. Alternatively, you should remove the commit hash completely for releases because this now looks a bit 'incomplete'. 🙈 Nonetheless, I would appreciate seeing the version number and commit hash.) It's still not in
|
@danielwelch should we close this based on #388 ? |
If someone can confirm it works :) |
Sorry, not working for me:
Addon version: 1.14.4.1 |
So you're not on edge? This is only supposed to work on edge as of now. |
Ah sry, yes didn't use edge 🙈 |
Finally got to install a dev instance with a separate CC2531. Seems the commit hash is still not working for edge.
Later edit: From |
@danielwelch apparently there is an error when running the pipeline. |
Fixed in 1.14.4.2 |
- Fixes #146, #229, #364 and #392 - Updates changelog - Add changelog for `edge` - Moves part of documentation in DOCS.md for `stable` - Adds Readme.md for `edge` - Added logo.png to display on add-on page automatically - Fixes add-on startup according to docs - Fixed commit unknown in plugin - Removed some centre styling for titles and added warning emoji - Removed some centre aligns for consistency - Simplified change log header and move details to root readme - Cleaned up change log completely to show only changes and move rest of meta to root readme - Updated change log with better details - Added docs for supporting new devices using new method - Updated changelog based on diffs in tags on the repo - Improved visibility of breaking changes in change log
I am using the latest zigbee2mqtt with this HassIO addon and observed that it always publishes something like
I am referring to the
"commit":"unknown"
here. Should be just a minor issue. Or do I miss something?The text was updated successfully, but these errors were encountered: