feat(psalm): Mark all usage of deprecated code as error#52860
feat(psalm): Mark all usage of deprecated code as error#52860provokateurin merged 1 commit intomasterfrom
Conversation
Signed-off-by: provokateurin <kate@provokateurin.de>
ChristophWurst
left a comment
There was a problem hiding this comment.
PR authors will hate Psalm and not the reviewer who points these out. I'm in 👍
nit: semantic commit is ci or chore but not feat as there is no new feature added to the product 🤓
Copied from nextcloud/server#52860 Signed-off-by: Jonas <jonas@freesources.org>
Copied from nextcloud/server#52860 Signed-off-by: Jonas <jonas@freesources.org>
|
This (and thus checked baselines) suppress for |
Yes, because lib/private needs to use the deprecated interfaces in a lot of cases (often the new implementation calls the deprecated one). |
Summary
Based on #52853.
The idea is to prevent any new code to use already deprecated stuff. It will not be possible everywhere since we sometimes have no replacement, but then it should just be added to the baseline.
This change adds a lot to the baseline, but it's basically just a todo list for technical debt cleanups :)
Checklist