1 min readSep 12, 2017
oh absolutely, good point. No PR should be looked at before all check and tests where run. Ideally that also includes UI smoke tests.
In addition one thing we do is: everyone can flag a PR as “needs to be manual checked” if he spots something that could potential be dangerous. Then it can not be merged before someone checked it out and run locally.