remove CargoTomlError

Author: xxchan

src/format_report_formatter.rs

@@ -146,8 +146,5 @@ fn error_kind_to_snippet_annotation_type(error_kind: &ErrorKind) -> AnnotationTy
         | ErrorKind::InvalidGlobPattern(_)
         | ErrorKind::VersionMismatch => AnnotationType::Error,
         ErrorKind::DeprecatedAttr => AnnotationType::Warning,
-        ErrorKind::CargoTomlError(_) => {
-            unreachable!("Cargo.toml formatting error is not contained in FormatReport.")
-        }
     }
 }

src/formatting/cargo_toml.rs

@@ -42,8 +42,8 @@ pub(crate) fn format_cargo_toml_inner(content: &str, config: &Config) -> Result<
 }
 
 impl From<TomlError> for ErrorKind {
-    fn from(e: TomlError) -> Self {
-        ErrorKind::CargoTomlError(format!("{e}"))
+    fn from(_: TomlError) -> Self {
+        ErrorKind::ParseError
     }
 }
 
@@ -135,9 +135,8 @@ impl VisitMut for SortKey {
                 let table = match section.as_table_mut() {
                     Some(table) => table,
                     None => {
-                        self.error = Some(ErrorKind::CargoTomlError(
-                            "package should be a table".into(),
-                        ));
+                        // package should be a table
+                        self.error = Some(ErrorKind::ParseError);
                         return;
                     }
                 };

src/lib.rs

@@ -133,8 +133,6 @@ pub enum ErrorKind {
     /// Invalid glob pattern in `ignore` configuration option.
     #[error("Invalid glob pattern found in ignore list: {0}")]
     InvalidGlobPattern(ignore::Error),
-    #[error("Error when formatting Cargo.toml: {0}")]
-    CargoTomlError(String),
 }
 
 impl ErrorKind {