Azure / azure-iot-sdk-node

A Node.js SDK for connecting devices to Microsoft Azure IoT services

https://docs.microsoft.com/en-us/azure/iot-hub/

Other

261 stars 227 forks source link

Mean time recovery #897

Closed YoDaMa closed 4 years ago

YoDaMa commented 4 years ago

Checklist

[ ] I have read the [contribution guidelines] (https://github.com/Azure/azure-iot-sdk-node/blob/master/.github/CONTRIBUTING.md).
[ ] I added or modified the existing tests to cover the change (we do not allow our test coverage to go down).
If this is a modification that impacts the behavior of a public API
- [ ] I edited the corresponding document in the devdoc folder and added or modified requirements.

Reference/Link to the issue solved with this PR (if any)

Description of the problem

Description of the solution

anthonyvercolano commented 4 years ago

!/usr/bin/env node

What is this?

Refers to: sdklab/longhaultests/src/iothub_longhaul.ts:1 in 84bae55. [](commit_id = 84bae551b204e3443bff401f25f6b817b30a003b, deletion_comment = False)

anthonyvercolano commented 4 years ago

Overall comment for the MTR. For the overall scheme about what is going on and how each part works is not documented very well.

Internally to each piece should be better documented about what it's individually trying to do to accomplish the overall task.

A much more extensive readme covering would be nice.

YoDaMa commented 4 years ago

/azp run