fix(installer): tolerate diagnostics during resolution
Publish UPM package / publish (push) Successful in 6s
Publish UPM package / publish (push) Successful in 6s
This commit is contained in:
@@ -82,7 +82,12 @@ namespace ShrinkSDK.Installer
|
||||
}
|
||||
else
|
||||
{
|
||||
SetStatus("Package diagnostics failed: " + request.Error.message, MessageType.Warning);
|
||||
var errorMessage = request.Error?.message;
|
||||
SetStatus(
|
||||
string.IsNullOrWhiteSpace(errorMessage)
|
||||
? "Package diagnostics did not complete while Unity was resolving packages. Refresh diagnostics after resolution completes."
|
||||
: "Package diagnostics failed: " + errorMessage,
|
||||
MessageType.Warning);
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -92,8 +97,7 @@ namespace ShrinkSDK.Installer
|
||||
try
|
||||
{
|
||||
ShrinkSdkManifestStore.EnsureShrinkSdkRegistry();
|
||||
Client.Resolve();
|
||||
SetStatus("Registry configuration written. Unity is resolving packages.", MessageType.Info);
|
||||
SetStatus("Registry configuration written. Unity will resolve the updated manifest.", MessageType.Info);
|
||||
}
|
||||
catch (Exception exception)
|
||||
{
|
||||
@@ -106,9 +110,8 @@ namespace ShrinkSDK.Installer
|
||||
try
|
||||
{
|
||||
ShrinkSdkManifestStore.EnsurePackageInstallation(entry);
|
||||
Client.Resolve();
|
||||
SetStatus(
|
||||
"Pinned " + entry.PackageName + " " + entry.Version + " in Packages/manifest.json. Unity is resolving packages.",
|
||||
"Pinned " + entry.PackageName + " " + entry.Version + " in Packages/manifest.json. Unity will resolve the updated manifest.",
|
||||
MessageType.Info);
|
||||
}
|
||||
catch (Exception exception)
|
||||
|
||||
Reference in New Issue
Block a user