Merge pull request #152 from mamba-org/feat/expose-convert-subdir-error

fix: expose error to outside crates
conda · Apr 3, 2023 · fade382 · fade382
2 parents bd4229f + 793fd2d
commit fade382
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/crates/rattler_conda_types/src/lib.rs b/crates/rattler_conda_types/src/lib.rs
@@ -33,7 +33,7 @@ pub use no_arch_type::{NoArchKind, NoArchType};
 pub use platform::{ParsePlatformError, Platform};
 pub use prefix_record::PrefixRecord;
 pub use repo_data::patches::{PackageRecordPatch, PatchInstructions, RepoDataPatch};
-pub use repo_data::{ChannelInfo, PackageRecord, RepoData};
+pub use repo_data::{ChannelInfo, ConvertSubdirError, PackageRecord, RepoData};
 pub use repo_data_record::RepoDataRecord;
 pub use run_export::RunExportKind;
 pub use version::{ParseVersionError, ParseVersionErrorKind, Version};

diff --git a/crates/rattler_conda_types/src/repo_data/mod.rs b/crates/rattler_conda_types/src/repo_data/mod.rs
@@ -180,15 +180,18 @@ impl RepoData {
 /// An error that can occur when parsing a platform from a string.
 #[derive(Debug, Error, Clone, Eq, PartialEq)]
 pub enum ConvertSubdirError {
+    /// No known combination for this platform is known
     #[error("platform: {platform}, arch: {arch} is not a known combination")]
     NoKnownCombination {
         /// The platform string that could not be parsed.
         platform: String,
         /// The architecture.
         arch: String,
     },
+    /// Platform key is empty
     #[error("platform key is empty in index.json")]
     PlatformEmpty,
+    /// Arc key is empty
     #[error("arch key is empty in index.json")]
     ArchEmpty,
 }