Skip to content

AppVeyor CI support #117

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

Closed
wants to merge 2 commits into from
Closed

AppVeyor CI support #117

wants to merge 2 commits into from

Conversation

FeodorFitsner
Copy link
Contributor

This PR adds support of AppVeyor CI.

Similar to TeamCity there is a custom Visitor implementation which is auto-detected or enforced with -appveyor command line switch. AppVeyorVisitor uses AppVeyor Build Worker API to communicate test results to AppVeyor real-time build console.

@bradwilson
Copy link
Member

This is merged in 17e4ac0. Please ensure that the code still works as intended.

We would also like to see more tests related to this. Is it possible to unit test the bits that talk to the AppVeyor API without setting up an AppVeyor account?

Also, do AppVeyor users want similar support for the MSBuild runner?

@FeodorFitsner
Copy link
Contributor Author

Thank you! I will test AppVeyor runner. We have the same functionality for MSBuild - it's a custom logger sending messages to AV build console.

I think it's possible to test AV integration without a real account if to make a fake implementation for REST API client. I will look into that.

@bradwilson bradwilson closed this Aug 2, 2014
@bradwilson
Copy link
Member

Please open a new PR if you have any further requests.

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