Updated Readme, changed PduLength to PDULength

mathiask88 / node-snap7

node.js wrapper for snap7

MIT License

163 stars 59 forks source link

Closed DJWassink closed 8 years ago

DJWassink commented 8 years ago

Just a small update to the readme PduLength seems to be PDULength

mathiask88 commented 8 years ago

Yes, you are right. Can you also change PduRequested to PDURequested and force push?

DJWassink commented 8 years ago

Aah right completely missed the PDURequested, added it

mathiask88 commented 8 years ago

Would be cool if you can sqash the two commits into one and give it a proper commit message.

DJWassink commented 8 years ago

Hope this is acceptable ;) thanks for your awesome work so far on this wrapper!

mathiask88 commented 8 years ago

Thank you very much for contributing! I'm very pleased to hear that and I hope the few guys using it are creating awesome stuff with it ;)