iotests/pylintrc: allow up to 10 similar lines

[PATCH] iotests/pylintrc: allow up to 10 similar lines

Vladimir Sementsov-Ogievskiy posted 1 patch 2 weeks, 3 days ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240430091329.2365053-1-vsementsov@yandex-team.ru

Maintainers: Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>

tests/qemu-iotests/pylintrc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Expand all Fold all

[PATCH] iotests/pylintrc: allow up to 10 similar lines

Posted by Vladimir Sementsov-Ogievskiy 2 weeks, 3 days ago

We want to have similar QMP objects in different tests. Reworking these
objects to make common parts by calling some helper functions doesn't
seem good. It's a lot more comfortable to see the whole QAPI request in
one place.

So, let's increase the limit, to unblock further commit
"iotests: add backup-discard-source"

Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
---

Hi all! That's a patch to unblock my PR
"[PULL 0/6] Block jobs patches for 2024-04-29"
  <20240429115157.2260885-1-vsementsov@yandex-team.ru>
  https://patchew.org/QEMU/20240429115157.2260885-1-vsementsov@yandex-team.ru/


 tests/qemu-iotests/pylintrc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/qemu-iotests/pylintrc b/tests/qemu-iotests/pylintrc
index de2e0c2781..05b75ee59b 100644
--- a/tests/qemu-iotests/pylintrc
+++ b/tests/qemu-iotests/pylintrc
@@ -55,4 +55,4 @@ max-line-length=79
 
 [SIMILARITIES]
 
-min-similarity-lines=6
+min-similarity-lines=10
-- 
2.34.1

Re: [PATCH] iotests/pylintrc: allow up to 10 similar lines

Posted by Philippe Mathieu-Daudé 2 weeks, 3 days ago

On 30/4/24 11:13, Vladimir Sementsov-Ogievskiy wrote:
> We want to have similar QMP objects in different tests. Reworking these
> objects to make common parts by calling some helper functions doesn't
> seem good. It's a lot more comfortable to see the whole QAPI request in
> one place.
> 
> So, let's increase the limit, to unblock further commit
> "iotests: add backup-discard-source"
> 
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
> ---
> 
> Hi all! That's a patch to unblock my PR
> "[PULL 0/6] Block jobs patches for 2024-04-29"
>    <20240429115157.2260885-1-vsementsov@yandex-team.ru>
>    https://patchew.org/QEMU/20240429115157.2260885-1-vsementsov@yandex-team.ru/
> 
> 
>   tests/qemu-iotests/pylintrc | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/qemu-iotests/pylintrc b/tests/qemu-iotests/pylintrc
> index de2e0c2781..05b75ee59b 100644
> --- a/tests/qemu-iotests/pylintrc
> +++ b/tests/qemu-iotests/pylintrc
> @@ -55,4 +55,4 @@ max-line-length=79
>   
>   [SIMILARITIES]
>   
> -min-similarity-lines=6
> +min-similarity-lines=10

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>