Skip to content

[WIP] Adding Flowdock support#5

Open
jsjohnst wants to merge 1 commit intojoonty:masterfrom
jsjohnst:add_flowdock_support
Open

[WIP] Adding Flowdock support#5
jsjohnst wants to merge 1 commit intojoonty:masterfrom
jsjohnst:add_flowdock_support

Conversation

@jsjohnst
Copy link

Still needs testing to make sure the format looks good, but wanted to push up a WIP pull request in case anyone else wants to try this.

@joonty
Copy link
Owner

joonty commented Nov 10, 2015

Hi, thanks very much for this!

I don't use Flowdock, but I'd be happy to merge it in considering the code looks good and this project is still pre version 1.0.0, as long as you're happy with it. If you want to go ahead and do any more testing, just let me know when you're happy and I'll merge it.

Thanks!

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to satisfy my OCD, could you get everything in the following hash indented with two spaces instead of a mixture of 2 and 3? Thanks!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Of course! Was annoyed by that too, but wanted to make the pull before I fell asleep. Will resolve while testing today

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jsjohnst, much appreciated!

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