You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The sys module (runtime) was deprecated ten years ago. Currently, all it does is reference the util module. After these ten years, it's time to remove it entirely (IMO).
Chances are, little systems are still using it (and the latest Node.js version).
I looked through the internals, sys isn't used anywhere where it can't be removed (tests).
I know it's been discussed in the past, but I believe that we shouldn't keep it for the sake of old systems, because chances are they aren't using the latest versions of node anyway
I still think we should discuss it's removal. It doesn't have any maintenance cost either way, but having it is redundant, and I doubt anyone using the latest Node.js version is also using the sys module.
After these ten years, it's time to remove it entirely (IMO).
Why? The cost of having it for backwards compatibility is small and breakage is non-zero. This issue is the most time we've spent on it in the last year probably.
The only maintenance cost of node:sys is those issues/PRs coming up about once per year suggesting we remove it 😅 I'm going to go ahead and close this issue in an effort to keep that maintenance burden low (I think your question have been answered – and if it hasn't, please read the linked PRs to get the context, you should find out that the previous objections still stand), see you next year everyone 🥲
Reacted by Aviv Keller, Moshe Atlow and Benjamin Gruenbaum
Activity
avivkeller commentedon Apr 24, 2024
This would be a
semver-majorchange if implemented.richardlau commentedon Apr 24, 2024
This has come up several times before. Every time we decided to leave things as-is. There's no benefit to removing it.
MoLow commentedon Apr 24, 2024
I recall running citgm had prevented removing this
avivkeller commentedon Apr 24, 2024
I looked through the internals,
sysisn't used anywhere where it can't be removed (tests).I know it's been discussed in the past, but I believe that we shouldn't keep it for the sake of old systems, because chances are they aren't using the latest versions of node anyway
richardlau commentedon Apr 24, 2024
Refs: #49520
avivkeller commentedon Apr 24, 2024
I still think we should discuss it's removal. It doesn't have any maintenance cost either way, but having it is redundant, and I doubt anyone using the latest Node.js version is also using the sys module.
benjamingr commentedon Apr 24, 2024
Why? The cost of having it for backwards compatibility is small and breakage is non-zero. This issue is the most time we've spent on it in the last year probably.
aduh95 commentedon Apr 24, 2024
The only maintenance cost of
node:sysis those issues/PRs coming up about once per year suggesting we remove it 😅 I'm going to go ahead and close this issue in an effort to keep that maintenance burden low (I think your question have been answered – and if it hasn't, please read the linked PRs to get the context, you should find out that the previous objections still stand), see you next year everyone 🥲