From d3e408556b8093c4c03a7471655069ba5cf698ba Mon Sep 17 00:00:00 2001 From: "patchback[bot]" <45432694+patchback[bot]@users.noreply.github.com> Date: Thu, 25 Jun 2026 14:07:10 -0400 Subject: [PATCH] ci: suppress SonarCloud python:S7498 for argument_spec dict() convention (#1152) (#1154) Ansible module argument_spec is conventionally defined with the dict(...) constructor rather than {...} literals, for readability and consistency across the collection. SonarCloud's python:S7498 ("Literal syntax should be preferred...") flags this as a false positive and re-fires on every new module argument. Ignore the rule project-wide via sonar-project.properties. Fixes #1151 (cherry picked from commit 1c20b53bdc40764c7638ac07266de912c8f5d12e) Co-authored-by: Yuriy Novostavskiy --- sonar-project.properties | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/sonar-project.properties b/sonar-project.properties index 39244934..f1246d96 100644 --- a/sonar-project.properties +++ b/sonar-project.properties @@ -12,3 +12,11 @@ sonar.python.version=3.12 sonar.newCode.referenceBranch=stable-5 sonar.exclusions=tests/**,.tox/** + +# Ansible module argument_spec is conventionally defined with the dict(...) +# constructor (not {...} literals) for readability and consistency across the +# collection. Suppress python:S7498 ("prefer literal syntax"), which conflicts +# with that convention and otherwise re-fires on every new module argument. +sonar.issue.ignore.multicriteria=e1 +sonar.issue.ignore.multicriteria.e1.ruleKey=python:S7498 +sonar.issue.ignore.multicriteria.e1.resourceKey=**/*.py