TypeRegistrar should be public

[scosec]

Information

• .Net Core
• Version: 6.0

Describe the bug
The typeregistrar should be a public class. If you follow the example from the documentation with dependency injection it doesn't work because the class is internal and can't be used outside the namespace.

To Reproduce
Copy the documentation example with dependency injection.

Expected behavior
Allow usage for the TypeRegistrar

Screenshots

[patriksvensson]

TypeRegistrar should not be made public. It's just used internally if no custom type-registrar is available. We should update the documentation.

[scosec]

Ah oke, I will create my own then! Thanks for the quick reply!

[nils-a]

@scosec, if you need a prebuild TypeRegistrar, you can have a look at

• https://www.nuget.org/packages/Spectre.Console.Extensions.Hosting,
• https://www.nuget.org/packages/Spectre.Console.Registrars.Microsoft-Di or
• https://www.nuget.org/packages/Spectre.Console.Registrars.SimpleInjector

[scosec]

Hi Nils,

Thanks for your reply. Certainly helpfull! :)

And a great project guys. I was building my own task runner and then heard this tool on the dotnet rocks podcast.
Let's say. Sold :)

[patriksvensson]

@scosec Thank you! What episode was that if I might ask?

[scosec]

@patriksvensson they mentioned it in the last episode wolverine.net command and messagebus with jeremy miller.