Skip to content

Commit 4123c06

Browse files
authored
Ensure consistent error message about missing YAML parser (#281)
Version requirement is ruamel >= 0.17
1 parent a23451d commit 4123c06

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

box/converters.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@
2929
except ImportError:
3030
pyyaml_available = False
3131

32+
MISSING_PARSER_ERROR = "No YAML Parser available, please install ruamel.yaml>=0.17 or PyYAML"
33+
3234
toml_read_library: Optional[Any] = None
3335
toml_write_library: Optional[Any] = None
3436
toml_decode_error: Optional[Callable] = None
@@ -196,7 +198,7 @@ def _to_yaml(
196198
elif pyyaml_available:
197199
return yaml.dump(obj, stream=f, default_flow_style=default_flow_style, **yaml_kwargs)
198200
else:
199-
raise BoxError("No YAML Parser available, please install ruamel.yaml>0.17 or PyYAML")
201+
raise BoxError(MISSING_PARSER_ERROR)
200202

201203
else:
202204
if ruamel_available:
@@ -210,7 +212,7 @@ def _to_yaml(
210212
elif pyyaml_available:
211213
return yaml.dump(obj, default_flow_style=default_flow_style, **yaml_kwargs)
212214
else:
213-
raise BoxError("No YAML Parser available, please install ruamel.yaml>0.17 or PyYAML")
215+
raise BoxError(MISSING_PARSER_ERROR)
214216

215217

216218
def _from_yaml(
@@ -237,7 +239,7 @@ def _from_yaml(
237239
kwargs["Loader"] = yaml.SafeLoader
238240
data = yaml.load(f, **kwargs)
239241
else:
240-
raise BoxError("No YAML Parser available, please install ruamel.yaml>0.15 or PyYAML")
242+
raise BoxError(MISSING_PARSER_ERROR)
241243
elif yaml_string:
242244
if ruamel_available:
243245
yaml_loader = YAML(typ=ruamel_typ)
@@ -249,7 +251,7 @@ def _from_yaml(
249251
kwargs["Loader"] = yaml.SafeLoader
250252
data = yaml.load(yaml_string, **kwargs)
251253
else:
252-
raise BoxError("No YAML Parser available, please install ruamel.yaml>0.17 or PyYAML")
254+
raise BoxError(MISSING_PARSER_ERROR)
253255
else:
254256
raise BoxError("from_yaml requires a string or filename")
255257
return data

0 commit comments

Comments
 (0)