Remember when I have been working in Ericsson we had been using a gerrit review and you simply
would not be able to merge anything until it has +2(r-b) and no tests failing. Or you have to exclude unstable tests at least until those are fixed. It was simply automated and no exceptions, no matter how good person you are ๐ Best Regards, Lisovskiy Stanislav Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo ________________________________ From: Hiler, Arkadiusz Sent: Wednesday, March 18, 2020 10:55:19 AM To: Lisovskiy, Stanislav Cc: intel-gfx@lists.freedesktop.org; Peres, Martin; Vudum, Lakshminarayana Subject: Re: โ Fi.CI.DOCS: warning for Consider DBuf bandwidth when calculating CDCLK (rev2) On Wed, Mar 18, 2020 at 10:43:29AM +0200, Lisovskiy, Stanislav wrote: > Wonder, how we end up merging _stuff_ with failed IGT and loads of warnings.. > > https://patchwork.freedesktop.org/series/74738/ > > ... while I get beaten for each and every single warning in my patches ๐ True. This shouldn't get merged like this. Ask the authors and the commiters why this got in without anyone looking at the docs issue and without any explanation of the failures in Fi.CI.IGT. -- Cheers, Arek
_______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
