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

fix for new tslint rules #264

Merged
merged 1 commit into from Feb 17, 2017

Conversation

Projects
None yet
3 participants
@lijiarui
Member

lijiarui commented Feb 17, 2017

@lijiarui

This comment has been minimized.

Show comment
Hide comment
@lijiarui

lijiarui Feb 17, 2017

Member

sorry for confusing you by the previous commit, because I cannot find all the modified code, so I create a new PR here.

Member

lijiarui commented Feb 17, 2017

sorry for confusing you by the previous commit, because I cannot find all the modified code, so I create a new PR here.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Feb 17, 2017

Coverage Status

Changes Unknown when pulling 487a06d on lijiarui:tslint into ** on wechaty:master**.

coveralls commented Feb 17, 2017

Coverage Status

Changes Unknown when pulling 487a06d on lijiarui:tslint into ** on wechaty:master**.

@zixia zixia merged commit 97be903 into Chatie:master Feb 17, 2017

6 checks passed

ci/circleci Your tests passed on CircleCI!
Details
codacy/pr Good work! A positive pull request.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls First build on master at 55.799%
Details
security/snyk No new vulnerabilities
Details
@zixia

This comment has been minimized.

Show comment
Hide comment
@zixia

zixia Feb 17, 2017

Member

Thank you for linting!

Member

zixia commented Feb 17, 2017

Thank you for linting!

@lijiarui lijiarui deleted the lijiarui:tslint branch Feb 17, 2017

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