refactor: drop the subplot::Result type alias

Replace subplot::Result<T> with Result<T, SubplotError>. I find this
now to be clearer, as I don't need to remind myself which Result is
being used where. This should not be a breaking change.

Sponsored-by: author

1 files changed, 2 insertions, 3 deletions

diff --git a/src/templatespec.rs b/src/templatespec.rs
index 5d7150b..dbb39dc 100644
--- a/src/templatespec.rs
+++ b/src/templatespec.rs
@@ -1,5 +1,4 @@
 use crate::resource;
-use crate::Result;
 use crate::SubplotError;
 
 use serde::Deserialize;
@@ -21,7 +20,7 @@ pub struct TemplateSpec {
 
 impl TemplateSpec {
     // Create a new TemplateSpec from YAML text.
-    fn from_yaml(yaml: &str) -> Result<TemplateSpec> {
+    fn from_yaml(yaml: &str) -> Result<TemplateSpec, SubplotError> {
         Ok(serde_yaml::from_str(yaml)?)
     }
 
@@ -40,7 +39,7 @@ impl TemplateSpec {
     }
 
     /// Read a template.yaml file and create the corresponding TemplateSpec.
-    pub fn from_file(filename: &Path) -> Result<TemplateSpec> {
+    pub fn from_file(filename: &Path) -> Result<TemplateSpec, SubplotError> {
         let yaml = resource::read_as_string(filename, None)?;
         let spec = TemplateSpec::from_yaml(&yaml)?;
         let dirname = match filename.parent() {