Recentemente me disseram que era uma prática ruim ter marcado vários métodos em nosso código com o[Obsolete] atributo. Esses métodos eram internos à nossa base de código, em vez de estarem em uma API. Os métodos manipularam uma função de criptografia mais antiga.

Eu senti que era uma maneira rápida e segura de denotar para o restante da equipe que esses métodos não deveriam ser usados e forneci uma mensagem para sugerir alternativas.

Outros achavam que eu deveria ter removido completamente os métodos, reescrevendo ou refatorando o código existente, conforme necessário. Além disso, considerou-se fácil demais ignorar os avisos do compilador.

Existe uma 'prática recomendada' para marcar código como obsoleto quando não estiver sendo usado por terceiros? Ou isso é amplamente subjetivo?