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
Added serverVersion to API; bumped API version to 2.2 #1064
Conversation
@@ -5,6 +5,7 @@ var hapLegacyTypes = require("hap-nodejs/accessories/types.js"); | |||
var log = require("./logger")._system; | |||
var User = require("./user").User; | |||
var PlatformAccessory = require("./platformAccessory").PlatformAccessory; | |||
var serverVersion = require("./version"); |
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.
I'm sorry but where is this serverVersion
thing?
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.
What do you mean by "is"? Where it's used? On line 27
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.
Nah never mind, I was thinking something else.
@@ -20,7 +21,10 @@ function API() { | |||
this._dynamicPlatforms = {}; // this._dynamicPlatforms[pluginName.platformName] = platform constructor | |||
|
|||
// expose the homebridge API version | |||
this.version = 2.1; | |||
this.version = "2.1.1"; |
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.
Please bump this to number 2.2 instead of a string of "2.1.1"
Alright, looks good to me :) |
Cool. Thanks! |
I'll try to see if there are any other changes going to happen, if not, I'm going to push this to npm probably later today. |
I'm extending https://github.com/gismo141/homebridge-server and would like to show the version of homebridge there by using the homebridge API.