Address pylint dangerous-default-value

.pylintrc

@@ -17,7 +17,6 @@ disable =
     line-too-long,
     # TODO(ssbarnea): remove temporary skips adding during initial adoption:
     consider-using-f-string,
-    dangerous-default-value,
     duplicate-code,
 
 [TYPECHECK]

src/ansiblelint/rules/__init__.py

@@ -270,10 +270,17 @@ def extend(self, more: List[AnsibleLintRule]) -> None:
         self.rules.extend(more)
 
     def run(
-        self, file: Lintable, tags: Set[str] = set(), skip_list: List[str] = []
+        self,
+        file: Lintable,
+        tags: Optional[Set[str]] = None,
+        skip_list: Optional[List[str]] = None,
     ) -> List[MatchError]:
         """Run all the rules against the given lintable."""
         matches: List[MatchError] = []
+        if tags is None:
+            tags = set()
+        if skip_list is None:
+            skip_list = []
 
         if not file.path.is_dir():
             try:

src/ansiblelint/runner.py

@@ -40,15 +40,20 @@ def __init__(
         *lintables: Union[Lintable, str],
         rules: "RulesCollection",
         tags: FrozenSet[Any] = frozenset(),
-        skip_list: List[str] = [],
-        exclude_paths: List[str] = [],
+        skip_list: Optional[List[str]] = None,
+        exclude_paths: Optional[List[str]] = None,
         verbosity: int = 0,
         checked_files: Optional[Set[Lintable]] = None
     ) -> None:
         """Initialize a Runner instance."""
         self.rules = rules
         self.lintables: Set[Lintable] = set()
 
+        if skip_list is None:
+            skip_list = []
+        if exclude_paths is None:
+            exclude_paths = []
+
         # Assure consistent type
         for item in lintables:
             if not isinstance(item, Lintable):