On 29 May 2011 19:13, Walter Bright <span dir="ltr"><<a href="mailto:walter@digitalmars.com">walter@digitalmars.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

For those doing compiler patches, I know it's a tough job, and thanks!<br>
<br>
But please consider running the dmd test suite (and try to build/run phobos unittests) before issuing a pull request. There are a lot of interdependencies in the compiler, and the test suite is designed to flush them out. The cool thing is the test suite consists of cases that are already minimized! The suite also is designed to not take that long to run.<br>


<br>
I run it constantly when I do dev on the compiler, and I constantly get dinged by it for something I overlooked. I consider the D test suite to be a major asset for us.<br>
<br>
Please use it. If you've got problems using it, please post here and we'll try to help.<br></blockquote><div><br></div><div>An idea relayed from Vladimir Panteleev: It could be a good idea to integrate pull requests with the automatic test runner - github has APIs for pull requests, the test runner could pull out the commits, test them, then post a comment on the pull request.</div>

<div><br></div><div>I see a few problems with this however:</div><div> - Added load for the test runner (although, they could be run with low priority and only when there aren't commits to master to test)</div><div> - Security - if you're testing all pull requests some sort of sandbox would be needed so the patch isn't running rm -rf or something more malicious</div>

<div> - More commits can be added to pull requests later on, it would need re-running in this case</div><div> - The test suite could be altered to make it look like the pull request is fine - although I guess anything that gets integrated will be looked over anyway, so this isn't much of an issue.</div>

<div> </div></div>-- <br>Robert<br><a href="http://octarineparrot.com/">http://octarineparrot.com/</a><br>