From ad1b9a870b5383ca9ada8c52b2bd76960d5579da Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Sun, 23 Aug 2020 14:22:27 -0400
Subject: move some console/logging logic out of SCore into a new LogManager

---
 src/SMAPI/Framework/IModMetadata.cs | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

(limited to 'src/SMAPI/Framework/IModMetadata.cs')

diff --git a/src/SMAPI/Framework/IModMetadata.cs b/src/SMAPI/Framework/IModMetadata.cs
index 1231b494..6a635b76 100644
--- a/src/SMAPI/Framework/IModMetadata.cs
+++ b/src/SMAPI/Framework/IModMetadata.cs
@@ -37,6 +37,9 @@ namespace StardewModdingAPI.Framework
         /// <summary>The reason the metadata is invalid, if any.</summary>
         string Error { get; }
 
+        /// <summary>A detailed technical message for <see cref="Error"/>, if any.</summary>
+        public string ErrorDetails { get; }
+
         /// <summary>Whether the mod folder should be ignored. This is <c>true</c> if it was found within a folder whose name starts with a dot.</summary>
         bool IsIgnored { get; }
 
@@ -65,8 +68,9 @@ namespace StardewModdingAPI.Framework
         /// <summary>Set the mod status.</summary>
         /// <param name="status">The metadata resolution status.</param>
         /// <param name="error">The reason the metadata is invalid, if any.</param>
+        /// <param name="errorDetails">A detailed technical message, if any.</param>
         /// <returns>Return the instance for chaining.</returns>
-        IModMetadata SetStatus(ModMetadataStatus status, string error = null);
+        IModMetadata SetStatus(ModMetadataStatus status, string error = null, string errorDetails = null);
 
         /// <summary>Set a warning flag for the mod.</summary>
         /// <param name="warning">The warning to set.</param>
-- 
cgit 


From 4088f4cb2bfe777cf6f86ac5fbf64f7d67565057 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Fri, 4 Sep 2020 22:02:59 -0400
Subject: simplify error shown for duplicate mods

---
 src/SMAPI/Framework/IModMetadata.cs | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

(limited to 'src/SMAPI/Framework/IModMetadata.cs')

diff --git a/src/SMAPI/Framework/IModMetadata.cs b/src/SMAPI/Framework/IModMetadata.cs
index 6a635b76..70cf0036 100644
--- a/src/SMAPI/Framework/IModMetadata.cs
+++ b/src/SMAPI/Framework/IModMetadata.cs
@@ -31,6 +31,9 @@ namespace StardewModdingAPI.Framework
         /// <summary>The metadata resolution status.</summary>
         ModMetadataStatus Status { get; }
 
+        /// <summary>The reason the mod failed to load, if applicable.</summary>
+        ModFailReason? FailReason { get; }
+
         /// <summary>Indicates non-error issues with the mod.</summary>
         ModWarning Warnings { get; }
 
@@ -65,12 +68,17 @@ namespace StardewModdingAPI.Framework
         /*********
         ** Public methods
         *********/
+        /// <summary>Set the mod status to <see cref="ModMetadataStatus.Found"/>.</summary>
+        /// <returns>Return the instance for chaining.</returns>
+        IModMetadata SetStatusFound();
+
         /// <summary>Set the mod status.</summary>
         /// <param name="status">The metadata resolution status.</param>
+        /// <param name="reason">The reason a mod could not be loaded.</param>
         /// <param name="error">The reason the metadata is invalid, if any.</param>
         /// <param name="errorDetails">A detailed technical message, if any.</param>
         /// <returns>Return the instance for chaining.</returns>
-        IModMetadata SetStatus(ModMetadataStatus status, string error = null, string errorDetails = null);
+        IModMetadata SetStatus(ModMetadataStatus status, ModFailReason reason, string error, string errorDetails = null);
 
         /// <summary>Set a warning flag for the mod.</summary>
         /// <param name="warning">The warning to set.</param>
-- 
cgit