Address pylint fixme (#2006)

.pylintrc

@@ -19,7 +19,6 @@ disable =
     consider-using-f-string,
     dangerous-default-value,
     duplicate-code,
-    fixme,
     protected-access,
 
 [TYPECHECK]

src/ansiblelint/cli.py

@@ -75,7 +75,7 @@ def load_config(config_file: str) -> Dict[Any, Any]:
         sys.exit(INVALID_CONFIG_RC)
 
     config["config_file"] = config_path
-    # TODO(ssbarnea): implement schema validation for config file
+    # See https://github.com/ansible-community/ansible-lint/issues/1803
     if isinstance(config, list):
         _logger.error(
             "Invalid configuration '%s', expected YAML mapping in the config file.",

src/ansiblelint/rules/__init__.py

@@ -125,8 +125,6 @@ def matchlines(self, file: "Lintable") -> List[MatchError]:
             matches.append(matcherror)
         return matches
 
-    # TODO(ssbarnea): Reduce mccabe complexity
-    # https://github.com/ansible-community/ansible-lint/issues/744
     def matchtasks(self, file: Lintable) -> List[MatchError]:
         matches: List[MatchError] = []
         if (

src/ansiblelint/skip_utils.py

@@ -110,9 +110,6 @@ def _append_skipped_rules(
             pyyaml_task_blocks = _get_task_blocks_from_playbook(pyyaml_data)
             ruamel_task_blocks = _get_task_blocks_from_playbook(ruamel_data)
         except (AttributeError, TypeError):
-            # TODO(awcrosby): running ansible-lint on any .yml file will
-            # assume it is a playbook, check needs to be added higher in the
-            # call stack, and can remove this except
             return pyyaml_data
     else:
         # For unsupported file types, we return empty skip lists

src/ansiblelint/yaml_utils.py

@@ -582,9 +582,10 @@ def __init__(
         self.Constructor = CustomConstructor
         self.Representer.add_representer(OctalIntYAML11, OctalIntYAML11.represent_octal)
 
-        # TODO: preserve_quotes loads all strings as a str subclass that carries
-        #       a quote attribute. Will the str subclasses cause problems in transforms?
-        #       Are there any other gotchas to this?
+        # We should preserve_quotes loads all strings as a str subclass that carries
+        # a quote attribute. Will the str subclasses cause problems in transforms?
+        # Are there any other gotchas to this?
+        #
         # This will only preserve quotes for strings read from the file.
         # anything modified by the transform will use no quotes, preferred_quote,
         # or the quote that results in the least amount of escaping.
@@ -656,7 +657,6 @@ def version(self, value: Optional[Union[str, Tuple[int, int]]]) -> None:
 
     def loads(self, stream: str) -> Any:
         """Load YAML content from a string while avoiding known ruamel.yaml issues."""
-        # TODO: maybe add support for passing a Lintable for the stream.
         if not isinstance(stream, str):
             raise NotImplementedError(f"expected a str but got {type(stream)}")
         text, preamble_comment = self._pre_process_yaml(stream)