fix: coerce plugin options to concrete types

mcbeet · Jul 1, 2021 · dd2de5a · dd2de5a
1 parent 3b6ea8d
commit dd2de5a
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 10 deletions.
diff --git a/beet/contrib/babelbox.py b/beet/contrib/babelbox.py
@@ -14,7 +14,7 @@
 
 import logging
 from csv import Dialect, DictReader, Sniffer
-from typing import Dict, Iterable, Optional, Type, Union
+from typing import Dict, List, Optional, Type, Union
 
 from pydantic import BaseModel
 
@@ -28,7 +28,7 @@
 
 
 class BabelboxOptions(BaseModel):
-    load: Iterable[str] = ()
+    load: List[str] = []
     dialect: Optional[str] = None
     filename_prefix: bool = False
 

diff --git a/beet/contrib/function_header.py b/beet/contrib/function_header.py
@@ -7,15 +7,15 @@
 ]
 
 
-from typing import Iterable, Optional
+from typing import List, Optional
 
 from pydantic import BaseModel
 
 from beet import Context, configurable
 
 
 class FunctionHeaderOptions(BaseModel):
-    match: Iterable[str] = ()
+    match: List[str] = []
     template: Optional[str] = "function_header.mcfunction"
 
 

diff --git a/beet/contrib/hangman.py b/beet/contrib/hangman.py
@@ -44,7 +44,7 @@
 
 
 class HangmanOptions(BaseModel):
-    match: Iterable[str] = ()
+    match: List[str] = []
 
 
 def beet_default(ctx: Context):

diff --git a/beet/contrib/load_yaml.py b/beet/contrib/load_yaml.py
@@ -9,7 +9,7 @@
 
 
 from dataclasses import InitVar, dataclass, field
-from typing import Any, Dict, Iterable, Optional, Type, TypeVar
+from typing import Any, Dict, List, Optional, Type, TypeVar
 
 import yaml
 from pydantic import BaseModel
@@ -30,8 +30,8 @@
 
 
 class LoadYamlOptions(BaseModel):
-    resource_pack: Iterable[str] = ()
-    data_pack: Iterable[str] = ()
+    resource_pack: List[str] = []
+    data_pack: List[str] = []
 
 
 def beet_default(ctx: Context):

diff --git a/beet/contrib/scoreboard.py b/beet/contrib/scoreboard.py
@@ -7,7 +7,7 @@
 ]
 
 
-from typing import Iterable
+from typing import List
 
 from pydantic import BaseModel
 
@@ -16,7 +16,7 @@
 
 class ScoreboardOptions(BaseModel):
     function: str = "scoreboard"
-    tags: Iterable[str] = ("minecraft:load",)
+    tags: List[str] = ["minecraft:load"]
 
 
 def beet_default(ctx: Context):