broidHQ / integrations

Connect your App to Multiple Messaging Channels with the W3C Open standard.
https://www.broid.ai
GNU Affero General Public License v3.0
746 stars 83 forks source link

Replace node-uuid with uuid #171

Closed chrisbodhi closed 7 years ago

chrisbodhi commented 7 years ago

Replace the deprecated node-uuid module with uuid.

Closes #108

And I've signed the CLA. My email is cboette@gmail.com

Please let me know if you have any questions, comments, or concerns -- thanks!

chrisbodhi commented 7 years ago

Build failed with a timeout:

> Install dependencies for broid-flowdock
yarn install v0.24.6
[1/4] Resolving packages...
[2/4] Fetching packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/flowdock/-/flowdock-0.9.1.tgz: ETIMEDOUT".
info If you think this is a bug, please open a bug report with the information provided in "/pipeline/source/broid-flowdock/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

And it looks like I'm not authorized to rebuild the project in Wercker.

dustinblackman commented 7 years ago

@chrisbodhi Thanks for this! I reran your tests, looks like you're failing lint at the moment. Let me know if you need some help.

chrisbodhi commented 7 years ago

@dustinblackman Thanks for catching that! Looks like it's passing now. Let me know if you need anything else.

killix commented 7 years ago

Tests pass. Ready to merge ;)