Skip to content

Commit

Permalink
Fixed flake8 lint errors
Browse files Browse the repository at this point in the history
  • Loading branch information
ansiblejunky committed Apr 6, 2021
1 parent 0a011ac commit b4b4d9d
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions src/ansiblelint/rules/VariableNamingRule.py
Original file line number Diff line number Diff line change
@@ -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
Expand Down Expand Up @@ -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'

Expand All @@ -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"
)
)
Expand Down Expand Up @@ -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"
)
)
Expand Down

0 comments on commit b4b4d9d

Please sign in to comment.