Re: New PR Review Process

2017-05-28 Thread Bryan Call
autest isn’t required for merges anymore, since the results of the tests are inconsistent. We have been using the CI system for awhile now to smoke test the PRs. The builds, regressions, clang-analyzer, and clang-format have been stable and rarely fail. If there is a problem with the CI in th

Re: New PR Review Process

2017-05-27 Thread James Peach
On 16 May 2017, at 19:42, Phil Sorber wrote: One of the things that was accomplished at the ATS Summits over the past several days (in addition to clearing up many many Coverity issues) was getting the build system integration with GitHub fixed up. We now have real commit status updates from