Skip to content
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 AppVeyor CI #200

Merged
merged 4 commits into from
Jan 21, 2018
Merged

Add AppVeyor CI #200

merged 4 commits into from
Jan 21, 2018

Conversation

mayeut
Copy link
Contributor

@mayeut mayeut commented Jan 21, 2018

I based this on latk/gcovr@772df78 as mentioned in #189
Working through all issues seen by the CI build except for one test which is now skipped when testing under windows: test_linked (test_gcovr.GcovrHtml)
Test results can be seen here: https://ci.appveyor.com/project/mayeut/gcovr/build/1.0.35

Copy link
Member

@latk latk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much for rebasing my initial attempts and getting everything to work! This is great :)
Now that we have automated tests on Windows, I feel more comfortable with higher-risk changes, e.g. refactoring such as in #43, and/or backporting changes from the dev branch.

@latk latk merged commit d80188c into gcovr:master Jan 21, 2018
@mayeut mayeut deleted the appveyor branch January 21, 2018 18:24
@latk latk mentioned this pull request Jan 22, 2018
@wolf99
Copy link

wolf99 commented Jan 22, 2018

Wooo, nice work guys!

JamesReynolds pushed a commit to JamesReynolds/gcovr that referenced this pull request Mar 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants