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

Update appveyor rule to support VS2017. #1356

Merged
merged 1 commit into from Sep 10, 2018

Conversation

Projects
None yet
4 participants
@gongminmin
Contributor

gongminmin commented Sep 10, 2018

Associated with #1294 .

@tencent-adm

This comment has been minimized.

Show comment
Hide comment
@tencent-adm

tencent-adm Sep 10, 2018

CLA assistant check
All committers have signed the CLA.

tencent-adm commented Sep 10, 2018

CLA assistant check
All committers have signed the CLA.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Sep 10, 2018

Coverage Status

Coverage remained the same at 99.922% when pulling a6be583 on gongminmin:AppVeyor2017 into f5f6052 on Tencent:master.

coveralls commented Sep 10, 2018

Coverage Status

Coverage remained the same at 99.922% when pulling a6be583 on gongminmin:AppVeyor2017 into f5f6052 on Tencent:master.

@miloyip miloyip merged commit c2aa79d into Tencent:master Sep 10, 2018

3 of 4 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 99.922%
Details
license/cla Contributor License Agreement is signed.
Details
@miloyip

This comment has been minimized.

Show comment
Hide comment
@miloyip

miloyip Sep 10, 2018

Collaborator

Thank you

Collaborator

miloyip commented Sep 10, 2018

Thank you

@gongminmin gongminmin deleted the gongminmin:AppVeyor2017 branch Sep 11, 2018

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