Re: [swift-dev] Initial smoke test PR message confusion

2016-07-29 Thread Michael Gottesman via swift-dev
I think we can do such a thing. +Mishal for a more definitive answer. The thing is that Mishal I think is trying to fix the {smoke,} test and merge and the normal test to set the smoke test flags so if one does: @swift-ci Please test and merge @swift-ci Please smoke test and merge swiftci can m

Re: [swift-dev] Initial smoke test PR message confusion

2016-07-29 Thread Ben Langmuir via swift-dev
Do we have control of this message? Can we change the heading to something like “Please run required checks”. > On Jul 28, 2016, at 10:40 PM, Michael Gottesman via swift-dev > wrote: > > Hey everyone. > > From talking with various people I think there is some confusion around the > initial

[swift-dev] Initial smoke test PR message confusion

2016-07-28 Thread Michael Gottesman via swift-dev
Hey everyone. >From talking with various people I think there is some confusion around the >initial message shown in the PR test state when one creates a PR. >Specifically, initially the PR test state says something like this: This does not mean that tests have started to be run automatically