[ds5678]

PR Details

This pull request deletes unused assembly processors.

• InteropProcessor
• RenameAssemblyProcessor

Both of these might have had some purpose in the past, but they currently have no effect on the build.

Related Issue

Part of #1835

Types of changes

• Docs change / refactoring / dependency upgrade

Checklist

• All new and existing tests passed.
• I have built and run the editor to try this change out.

Note

This conflicts with #3593 . I'll rebase after one of them is merged.

[Kryptos-FR]

The rename feature might be useful. Just because it isn't used now doesn't mean it won't in the future. It was primarly used when Xenko became Stride.

[ds5678]

The rename feature might be useful. Just because it isn't used now doesn't mean it won't in the future. It was primarly used when Xenko became Stride.

I'm not sure in what context this would be used, especially since you can change the name with AssemblyName in csproj files. Can you give an example?

[xen2]

About the rename feature: I think it's fine to remove it. Nothing in the build passes --rename-assembly anymore, AssemblyName in the csproj covers the normal case, and if we ever need it again for something like the Xenko -> Stride rename, we'll figure another way since the AssemblyProcessor will likely be gone.