-
Notifications
You must be signed in to change notification settings - Fork 178
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
Develop query #15
Develop query #15
Conversation
updated tests&README accordingly increased timeout for deleteDatabase() test
…ncreased timeout in tests for deleteDatabase()
Ok, I've added a warning and the options now have a "devmode" flag. Hope that does the job. let me know what you think. |
no need to be sorry, I'm gonna change it :) |
I hope you don't mind me using yoda conditions. |
…development', NODE_ENV == undefined. Default log function (console.log) can be overidden
request({ | ||
url: this.url('db/' + this.options.database + '/series', { q: query }), | ||
json: true | ||
}, this._parseCallback(callback)); | ||
}; | ||
|
||
// legacy function | ||
InfluxDB.prototype.readPoints = function(query, callback) { | ||
if (false !== this.options.depreciatedLogging) this.options.depreciatedLogging('influx.readPoints() has been depreciated, please use influx.query()'); |
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.
if ( false !== this.options.depreciatedLogging)
Just that one change, then I'm happy for you to merge this into master! |
ok, I removed it. Would you also bump the version afterwards? |
Hm...the failed build looks like an issue with travis to me |
Yep it was a Travis issue. It's due to the reporting of coverage to Coveralls.io which I should probably put as an after_script. |
Added `query()` and depreciated `readPoints()`
renamed readpoints() to query()