Make some ExtensionMethods public for ICustomRule about netarchtest.enhancededition HOT 2 CLOSED

What about going in the opposite direction and making Type TypeDefinitionExtensions.ToType(this TypeDefinition typeDefinition) public? Would not be that enough?

I would rather prefer to embrace end-user to use reflection API, instead of Mono.Cecil API which should be an implementation detail hidden from the end-user, but unfortunately this is a way that the original library has taken.

Regarding the second question, it is not possible in the current API, but it is fully supported internally, so the only question is how to expose that through API.

from netarchtest.enhancededition.

What about going in the opposite direction and making Type TypeDefinitionExtensions.ToType(this TypeDefinition typeDefinition) public? Would not be that enough?

I would rather prefer to embrace end-user to use reflection API, instead of Mono.Cecil API which should be an implementation detail hidden from the end-user, but unfortunately this is a way that the original library has taken.

Sure, I can live with that as well and was initially what I've been looking for. :-)

Regarding the second question, it is not possible in the current API, but it is fully supported internally, so the only question is how to expose that through API.

I guess, ICustomRule would need to be anyway redesigned, so that it does not have a visible Mono.Cecil dependency.

I would expect something like a CustomRuleResult.

return new CustomRuleResult(success: true, explanation: "Because of this and that

or

return CustomRuleResult.Success() and CustomRuleResult.Failed("Because of this and that")

But encapsulating all of stuff 'Mono.Cecil` or TypeDefinition and TypeReference would be a lot of work, I guess.

from netarchtest.enhancededition.