Would a partial disable based on `ExtensionState.DISABLING` which only reverts patches be permissible?

Thanks for the explanation. I don’t think anyone has considered that case so far (at least not that I’m aware).

I don’t see how this could be addressed any other way than banning optional overrides in the review guidelines. That seems like a good idea to me to avoid potential conflicts, in particular as extensions can easily move the condition check into the override.

@JustPerfection what do you think?

1 Like