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, 5 insertions, 2 deletions

diff --git a/src/steps.rs b/src/steps.rs
index c8c1bf6..ccbc588 100644
--- a/src/steps.rs
+++ b/src/steps.rs
@@ -1,4 +1,4 @@
-use crate::{Result, SubplotError};
+use crate::SubplotError;
 use serde::{Deserialize, Serialize};
 use std::fmt;
 
@@ -47,7 +47,10 @@ impl ScenarioStep {
     ///
     /// If the step uses the "and" or "but" keyword, use the default
     /// step kind instead.
-    pub fn new_from_str(text: &str, default: Option<StepKind>) -> Result<ScenarioStep> {
+    pub fn new_from_str(
+        text: &str,
+        default: Option<StepKind>,
+    ) -> Result<ScenarioStep, SubplotError> {
         let mut words = text.split_whitespace();
 
         let keyword = match words.next() {