Skip to content
This repository was archived by the owner on Sep 16, 2020. It is now read-only.

Conversation

@DovOps
Copy link
Collaborator

@DovOps DovOps commented Mar 14, 2018

#106 - Added support for signals client. NOTE: Advanced responses for subscribe/unsubscribe operations are passing direct API response payloads back.

@DovOps DovOps requested a review from ftbb March 14, 2018 00:01
@DovOps
Copy link
Collaborator Author

DovOps commented Mar 14, 2018

Thanks @pmonks your fix fixed the build in this PR. @pmonks @ftbb any ideas why CLA isnt working here? otherwise ready to merge.

@pmonks
Copy link
Contributor

pmonks commented Mar 14, 2018

@ftbb just a head's up that the failure report from cla-bot is incorrect - @Dovie does indeed have a CLA with the Foundation, so you can merge this PR if you're otherwise happy with it.

I'm raising a high priority issue with @maoo and @Kabrakssis for them to look at the clabot tomorrow - it seems to be struggling at the moment.

@ftbb ftbb merged commit be0bce6 into symphonyoss:develop Mar 14, 2018
@maoo
Copy link
Contributor

maoo commented Mar 15, 2018

Sorry for the problems caused by cla-bot; I checked the logs and it seems that it's a known issue https://github.com/ColinEberhardt/cla-bot/issues/74 .

While I work on a fix, @Dovie , could you please check if the email of your git client matches the one of your GitHub account? see octokit/octokit.net#1433 (comment) . TY. /CC @pmonks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants