doc: remove version from maintaining-dependencies.md#51195
doc: remove version from maintaining-dependencies.md#51195nodejs-github-bot merged 5 commits intonodejs:mainfrom
maintaining-dependencies.md#51195Conversation
Those create unreasonable maintenance burden as updating one dependency will conflict with another one. It's unclear we get much value out from duplicating this information in this document, so this commit removes it
|
Review requested:
|
|
I have mixed feelings, the maintainance burden is annoying but not having the dependency version tracked in a readable way sounds bad too. |
|
I think the point of having a workflow that automatically updates our deps is to stay up-to-date, and not burn out our volunteers' time with busy work, and if you look at the number of open PRs for dependency updates, you can see we're falling behind – because no one wants to do the work of fixing conflicts, I myself regret spending the time I spent on this when I see that as soon as one PR has landed, it has undone the work I just did by creating new conflicts.
OK but not landing the upgrade PRs is worse I think, and that's the current situation. The current situation has to change if you want me to keep interacting with those upgrade PRs, it's simply not worth my time to try to fix an endless flow of git conflicts – and more importantly, it's not worth releasers' time either. If you want a readable way to get the information, hopefully using git is enough? For example, if you go to https://github.com/nodejs/node/tree/main/deps, GitHub web UI will list the last commit and that's often the last update with the version number: |
targos
left a comment
There was a problem hiding this comment.
I already spent a long time (accumulated) fixing conflicts in LTS
|
Landed in 898149f |
Those create unreasonable maintenance burden as updating one dependency will conflict with another one. It's unclear we get much value out from duplicating this information in this document, so this commit removes it PR-URL: nodejs#51195 Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
mhdawson
left a comment
There was a problem hiding this comment.
LGTM. I think this made sense when we were doing it manually, but now that we have automation doing the updates the conflicts are not worty the overhead.
Those create unreasonable maintenance burden as updating one dependency will conflict with another one. It's unclear we get much value out from duplicating this information in this document, so this commit removes it PR-URL: #51195 Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Those create unreasonable maintenance burden as updating one dependency will conflict with another one. It's unclear we get much value out from duplicating this information in this document, so this commit removes it PR-URL: #51195 Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Those create unreasonable maintenance burden as updating one dependency will conflict with another one. It's unclear we get much value out from duplicating this information in this document, so this PR removes it.