process,doc: add missing deprecation code#37091
process,doc: add missing deprecation code#37091cjihrig wants to merge 1 commit intonodejs:masterfrom
Conversation
|
Fast-track? |
|
Landed in 4e833b6 |
|
Arg... thanks. I keep forgetting that the |
Lines 933 to 937 in c9992a0 I imagine it just needs something like this. |
Or maybe we could stop using |
nodejs/node-core-utils#420 was supposed to detect these. |
Key challenge with that is maintaining the order on landing (e.g. if I open a semver-major deprecation PR today that takes three months to land, but three other semver-minor doc only deprecations happen in the meantime... which has happened before). The one thing we could do is move away from numbered deprecation codes at all and move to a non-numeric code, e.g. |
My experience with PR for deprecation that stay open for a long time is that deprecations that are added in the mean time always create a git conflict anyway. So the PR author has to rebase to fix the git conflict, they may as well update the deprecation code. If you add to that a lint rule which would pick up duplicate deprecation codes, I think we would have a way better system that we have now. |
Refs: #36902