Hace poco me dijeron que era una mala práctica haber marcado varios métodos en nuestro código con el[Obsolete] atributo. Estos métodos eran internos de nuestra base de código, en lugar de estar en una API. Los métodos manejan una función de encriptación más antigua.

Sentí que era una forma rápida y segura de denunciar al resto del equipo que estos métodos no deberían usarse, y proporcioné un mensaje para sugerir alternativas.

Otros consideraron que debería haber eliminado los métodos por completo, reescribiendo o refactorizando el código existente según fuera necesario. Además, se pensó que era demasiado fácil pasar por alto las advertencias del compilador.

¿Existe una 'mejor práctica' para marcar el código como Obsoleto cuando no está siendo utilizado por terceros? ¿O es esto en gran medida subjetivo?