Re: Remove unnecessary 'always:' from CompilerWarnings task

2023-11-08 Thread Nazir Bilal Yavuz
Hi, Thanks for the review. On Wed, 8 Nov 2023 at 10:31, Peter Eisentraut wrote: > > On 05.09.23 12:25, Nazir Bilal Yavuz wrote: > > There are multiple 'always:' keywords under the CompilerWarnings task. > > Instead of that, we can use one 'always:' and move the instructions > > under this. So, I

Re: Remove unnecessary 'always:' from CompilerWarnings task

2023-11-07 Thread Peter Eisentraut
On 05.09.23 12:25, Nazir Bilal Yavuz wrote: There are multiple 'always:' keywords under the CompilerWarnings task. Instead of that, we can use one 'always:' and move the instructions under this. So, I removed unnecessary ones and rearranged indents according to that change. I'm not sure this ch

Remove unnecessary 'always:' from CompilerWarnings task

2023-09-05 Thread Nazir Bilal Yavuz
ted. Regards, Nazir Bilal Yavuz Microsoft From 3bd8f6ff5b8add90dcfe42761a3c1fe81a5c174a Mon Sep 17 00:00:00 2001 From: Nazir Bilal Yavuz Date: Tue, 5 Sep 2023 13:04:24 +0300 Subject: [PATCH v1] Remove unnecessary 'always:' from CompilerWarnings task Warning scripts and upload_caches ins