Closed tillig closed 11 months ago
Patch coverage: 92.68%
and project coverage change: -0.02%
:warning:
Comparison is base (
dc4e4cd
) 78.49% compared to head (36da90d
) 78.47%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
RegisterTypes
so if "invalid" types (types that generally can't be created via reflection - interfaces, abstract classes, etc.) get passed in, they will be ignored and not registered.RegisterType(Type t)
andRegisterType<T>()
calls so if someone tries to use a type in there that generally can't be created via reflection, anArgumentException
will come up and explain the issue rather than waiting for the container build to occur and hitting a "no constructors found" exception.parameterised
=>parameterized
. Given the fallback language on the assemblies isen-US
it seemed like fixing up docs and internal naming to also been-US
was reasonable. Also allows spell checkers set to that dictionary to find fewer things.ctxt
=>context
. Another spell checker issue. Community practice generally either usesctx
orcontext
, with most .NET code in the framework being a fully spelled-outcontext
. Updated references to go with the fully spelled-out version.I'll add some comments to the important bits for extra attention.