comtypes
comtypes copied to clipboard
enthought
Continuous integration with AppVeyor
What do you think about setting up continuous integration using AppVeyor? They support Windows and have a pretty nice build environment: https://www.appveyor.com/docs/build-environment/
It would be good for all merge requests/commits to automatically run the test suite, and it should also be somewhat simple to set up - but the proof will be in the pudding if the tests pass!
I'm in favor of this, but have not had the time to set it up. As for the tests, I know that there are memory leak tests that fail periodically. (I'm not actually sure if this is due to faulty tests or actual memory leaks.)
I can make a merge request with the files to set this up if you link this repository with AppVeyor? As you have access to it you should be able to sign up
On 21 Mar 2017 12:19, "Chris Farrow" [email protected] wrote:
I'm in favor of this, but have not had the time to set it up. As for the tests, I know that there are memory leak tests that fail periodically. (I'm not actually sure if this is due to faulty tests or actual memory leaks.)
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/enthought/comtypes/issues/120#issuecomment-288061602, or mute the thread https://github.com/notifications/unsubscribe-auth/AA-sh6jy2l7f0OZF-r7Q-OMOHTjxYybNks5rn8BRgaJpZM4MjtXt .
Awesome, thank you, I'll make a MR soon with at least some sanity checks in it.
Hey @cfarrow, can you check that you've enabled the webhook for 'pull requests'? My branch doesn't seem to be running.
Like so:
Probably not. I look to be locked out of the configuration screen. I'll get my privileges updated and then look into it.
AppVeyor CI is currently working fine and is helping us in our development.
@orf @cfarrow if the issue remains, please re-open.
