Skip to content

Commit de1a6df

Browse files
authored
Merge pull request #9413 from MaxWang-MS/ci_ar_fix
Add Scene Understanding to the Asset Retargeter ignore list
2 parents b2a1327 + cf564ae commit de1a6df

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Assets/MRTK/Tools/MSBuild/Scripts/AssetScriptReferenceRetargeter.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -376,8 +376,8 @@ private static bool ProcessYamlFile(string filePath, string targetPath, Dictiona
376376
// The OculusProfileGUID bypasses throwing the exception. The reason for this is that there is currently an asset (DefaultOculusXRSDKDeviceManagerProfile.asset) that is reliant
377377
// on Unity 2019+ specific code (OculusXRSDKDeviceManagerProfile.cs), which causes CI to fail since it's running on Unity 2018.
378378

379-
// Also bypass this exception for scripts in an InteractiveElement directory as those files are only supported in Unity 2019.
380-
if (id != OculusProfileGUID && !filePath.Contains("InteractiveElement"))
379+
// Also bypass this exception for scripts in the InteractiveElement and SceneUnderstanding directories as those files are only supported in Unity 2019.
380+
if (id != OculusProfileGUID && !filePath.Contains("InteractiveElement") && !filePath.Contains("SceneUnderstanding"))
381381
{
382382
throw new InvalidDataException($"A script without a class ({NonClassDictionary[id]}) is being processed on {Path.GetFileName(filePath)}.");
383383
}

0 commit comments

Comments
 (0)