Closed Elscrux closed 1 year ago
The new static abstract IAssetType seems to have an issue I didn't think of before. More details can be found here. https://github.com/dotnet/csharplang/issues/5955
The result is that trying to create any data structure like this is impossible with the static abstract implementation of Instances.
static abstract IAssetType Instance { get; } ... List<IAssetType> AssetTypes; // error (CS8920)
This PR is supposed to fix this issue by making IAssetType's Instance (temporarily) virtual and adding tests to ensure everyone implements a valid Instance.
Bug
The new static abstract IAssetType seems to have an issue I didn't think of before. More details can be found here. https://github.com/dotnet/csharplang/issues/5955
The result is that trying to create any data structure like this is impossible with the static abstract implementation of Instances.
PR
This PR is supposed to fix this issue by making IAssetType's Instance (temporarily) virtual and adding tests to ensure everyone implements a valid Instance.