From b4b4d9ddc1ef68ecf682306c25beeac4ad1e9eff Mon Sep 17 00:00:00 2001 From: John Date: Tue, 6 Apr 2021 15:48:29 -0700 Subject: [PATCH] Fixed flake8 lint errors --- src/ansiblelint/rules/VariableNamingRule.py | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/src/ansiblelint/rules/VariableNamingRule.py b/src/ansiblelint/rules/VariableNamingRule.py index ab37712c82..4fef483a52 100644 --- a/src/ansiblelint/rules/VariableNamingRule.py +++ b/src/ansiblelint/rules/VariableNamingRule.py @@ -1,5 +1,6 @@ -from typing import List, Any, Dict, Union - +from typing import TYPE_CHECKING, List, Any, Dict, Union +if TYPE_CHECKING: + from ansiblelint.constants import odict from ansiblelint.errors import MatchError from ansiblelint.file_utils import Lintable from ansiblelint.rules import AnsibleLintRule @@ -27,7 +28,7 @@ class VariableNamingRule(AnsibleLintRule): base_msg = 'All variables should be named using only lowercase and underscores' shortdesc = base_msg description = 'All variables should be named using only lowercase and underscores' - severity = 'MEDIUM' # ansible-lint displays severity when with --parseable-severity option + severity = 'MEDIUM' # ansible-lint displays severity when with --parseable-severity option tags = ['formatting', 'readability'] version_added = 'v5.0.5' @@ -54,8 +55,8 @@ def matchplay(self, file: "Lintable", data: "odict[str, Any]") -> List["MatchErr if is_invalid_variable_name(key): results.append( self.create_matcherror( - filename=file, - linenumber=vars['__line__'], + filename=file, + linenumber=vars['__line__'], message="Play defines variable '" + key + "' within 'vars' section that violates variable naming standards" ) ) @@ -98,8 +99,8 @@ def matchyaml(self, file: Lintable) -> List["MatchError"]: if is_invalid_variable_name(key): results.append( self.create_matcherror( - filename=file, - # linenumber=vars['__line__'], + filename=file, + # linenumber=vars['__line__'], message="File defines variable '" + key + "' that violates variable naming standards" ) )