search

Cloud-Automation / node-modbus

Modbus TCP Client/Server implementation for Node.JS
466 stars 174 forks source link

Tcp server #158

Closed AnthonyMujic closed 6 years ago

AnthonyMujic commented 6 years ago

Modified read-coils to cope with requests for less than a whole byte. The remaining bits are padded with zeros. Let me know if this is going down the wrong path.

AnthonyMujic commented 6 years ago

I notice I now have the option to 'Merge Pull Request'. I'm not sure of the etiquette - is it ok for me to merge my own pull requests into the v3 branch, or should someone still review it first?

stefanpoeter commented 6 years ago

Since this is a development branch you can merge it yourself. The whole Branche needs to be properly tested and reviewed before it is going to the main branch and being released.