New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Hot Listener Support (#820) #841

Merged
merged 11 commits into from Oct 2, 2017

Conversation

@zixia
Member

zixia commented Oct 1, 2017

@zixia zixia requested a review from lijiarui as a code owner Oct 1, 2017

@zixia

This comment has been minimized.

Show comment
Hide comment
@zixia

zixia Oct 2, 2017

Member

Ah, I'm in a hurry to use this PR with @binsee 's last code.

Please leave comments in issue #820 and let me know any suggestions.

Member

zixia commented Oct 2, 2017

Ah, I'm in a hurry to use this PR with @binsee 's last code.

Please leave comments in issue #820 and let me know any suggestions.

@zixia zixia merged commit c47715b into master Oct 2, 2017

8 of 9 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
ci/circleci Your tests passed on CircleCI!
Details
codacy/pr Good work! A positive pull request.
Details
codeclimate All good!
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.5%) to 54.571%
Details
security/snyk No new issues
Details

@zixia zixia deleted the hot-listener branch Oct 4, 2017

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