Skip to content

Conversation

@snowdd1
Copy link
Contributor

@snowdd1 snowdd1 commented Jan 22, 2017

third parameter doNotCloseConnection {Boolean} allows client program to
send multiple telegrams using the same connection.

See #43

Additionally: formatting, commenting (JSDoc) for functions

third parameter doNotCloseConnection {Boolean} allows client program to
send multiple telegrams using the same connection.

See #43

Additionally: formatting, commenting (JSDoc) for functions
@snowdd1
Copy link
Contributor Author

snowdd1 commented Jan 22, 2017

This should pin the physical KNX address with latest knxd as discussed in knxd/knxd#190 (comment)

@andreek
Copy link
Owner

andreek commented Feb 1, 2017

I don't like 4 space indention in node/js. Could you revert this change?

I tried to follow this guide.

@snowdd1
Copy link
Contributor Author

snowdd1 commented Feb 1, 2017

Of course.

@snowdd1
Copy link
Contributor Author

snowdd1 commented Feb 1, 2017

Ok, already fixed that one: http://nodeguide.com/style.html#variable-declarations

set to http://nodeguide.com/style.html settings, espc indentation 2 spc
instead of TAB
@andreek andreek merged commit cd62776 into andreek:master Feb 3, 2017
@snowdd1 snowdd1 deleted the doNotAutoclose branch February 3, 2017 10:21
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