-
Notifications
You must be signed in to change notification settings - Fork 29.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tools: move update-undici.sh to dep_updaters and create brotli maintaining md #47380
tools: move update-undici.sh to dep_updaters and create brotli maintaining md #47380
Conversation
Review requested:
|
f89a376
to
3fb789c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to either add spaces or remove the empty line so the code snippet is inside the list.
2fd443c
to
335a979
Compare
Can you please make the commit message clearer? |
1e1497a
to
a41e4f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The title says maintaining undici, but you are adding maintaining-brotli as well. Could you please adjust it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Commit Queue failed- Loading data for nodejs/node/pull/47380 ✔ Done loading data for nodejs/node/pull/47380 ----------------------------------- PR info ------------------------------------ Title tools: move update-undici.sh to dep_updaters and create brotli maintaining md (#47380) Author Marco Ippolito (@marco-ippolito) Branch marco-ippolito:feat/improve-deps-update -> nodejs:main Labels meta, tools, author ready, commit-queue-rebase Commits 2 - doc: create maintaining-brotli.md - tools: move update-undici.sh to dep_updaters and create maintaining-u… Committers 1 - Marco Ippolito PR-URL: https://github.com/nodejs/node/pull/47380 Refs: https://github.com/nodejs/security-wg/issues/828 Reviewed-By: Yagiz Nizipli Reviewed-By: Paolo Insogna Reviewed-By: Rafael Gonzaga Reviewed-By: Luigi Pinca ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/47380 Refs: https://github.com/nodejs/security-wg/issues/828 Reviewed-By: Yagiz Nizipli Reviewed-By: Paolo Insogna Reviewed-By: Rafael Gonzaga Reviewed-By: Luigi Pinca -------------------------------------------------------------------------------- ℹ This PR was created on Mon, 03 Apr 2023 09:59:54 GMT ✔ Approvals: 4 ✔ - Yagiz Nizipli (@anonrig): https://github.com/nodejs/node/pull/47380#pullrequestreview-1369790275 ✔ - Paolo Insogna (@ShogunPanda): https://github.com/nodejs/node/pull/47380#pullrequestreview-1376060783 ✔ - Rafael Gonzaga (@RafaelGSS) (TSC): https://github.com/nodejs/node/pull/47380#pullrequestreview-1376541837 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/47380#pullrequestreview-1378029737 ✔ Last GitHub CI successful ℹ Green GitHub CI is sufficient -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/main up to date... From https://github.com/nodejs/node * branch main -> FETCH_HEAD ✔ origin/main is now up-to-date - Downloading patch for 47380 From https://github.com/nodejs/node * branch refs/pull/47380/merge -> FETCH_HEAD ✔ Fetched commits as 9d9cf11a4bf8..a41e4f114fee -------------------------------------------------------------------------------- [main 0f59d9b17f] doc: create maintaining-brotli.md Author: Marco Ippolito Date: Mon Apr 3 10:50:40 2023 +0200 1 file changed, 26 insertions(+) create mode 100644 doc/contributing/maintaining-brotli.md Auto-merging .github/workflows/tools.yml [main 196d1ff98b] tools: move update-undici.sh to dep_updaters and create maintaining-undici.md Author: Marco Ippolito Date: Mon Apr 3 11:52:59 2023 +0200 6 files changed, 56 insertions(+), 20 deletions(-) create mode 100644 doc/contributing/maintaining-undici.md rename tools/{ => dep_updaters}/update-undici.sh (60%) ✔ Patches applied There are 2 commits in the PR. Attempting autorebase. Rebasing (2/4)https://github.com/nodejs/node/actions/runs/4660665489 |
Commit Queue failed- Loading data for nodejs/node/pull/47380 ✔ Done loading data for nodejs/node/pull/47380 ----------------------------------- PR info ------------------------------------ Title tools: move update-undici.sh to dep_updaters and create brotli maintaining md (#47380) Author Marco Ippolito (@marco-ippolito) Branch marco-ippolito:feat/improve-deps-update -> nodejs:main Labels meta, tools, author ready Commits 2 - doc: create maintaining-brotli.md - tools: move update-undici.sh to dep_updaters and create maintaining-u… Committers 1 - Marco Ippolito PR-URL: https://github.com/nodejs/node/pull/47380 Refs: https://github.com/nodejs/security-wg/issues/828 Reviewed-By: Yagiz Nizipli Reviewed-By: Paolo Insogna Reviewed-By: Rafael Gonzaga Reviewed-By: Luigi Pinca ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/47380 Refs: https://github.com/nodejs/security-wg/issues/828 Reviewed-By: Yagiz Nizipli Reviewed-By: Paolo Insogna Reviewed-By: Rafael Gonzaga Reviewed-By: Luigi Pinca -------------------------------------------------------------------------------- ℹ This PR was created on Mon, 03 Apr 2023 09:59:54 GMT ✔ Approvals: 4 ✔ - Yagiz Nizipli (@anonrig): https://github.com/nodejs/node/pull/47380#pullrequestreview-1369790275 ✔ - Paolo Insogna (@ShogunPanda): https://github.com/nodejs/node/pull/47380#pullrequestreview-1376060783 ✔ - Rafael Gonzaga (@RafaelGSS) (TSC): https://github.com/nodejs/node/pull/47380#pullrequestreview-1376541837 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/47380#pullrequestreview-1378029737 ✔ Last GitHub CI successful ℹ Green GitHub CI is sufficient -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/main up to date... From https://github.com/nodejs/node * branch main -> FETCH_HEAD ✔ origin/main is now up-to-date - Downloading patch for 47380 From https://github.com/nodejs/node * branch refs/pull/47380/merge -> FETCH_HEAD ✔ Fetched commits as 97d3912eb8dd..a41e4f114fee -------------------------------------------------------------------------------- [main d8befc0c24] doc: create maintaining-brotli.md Author: Marco Ippolito Date: Mon Apr 3 10:50:40 2023 +0200 1 file changed, 26 insertions(+) create mode 100644 doc/contributing/maintaining-brotli.md Auto-merging .github/workflows/tools.yml [main ef512cffd6] tools: move update-undici.sh to dep_updaters and create maintaining-undici.md Author: Marco Ippolito Date: Mon Apr 3 11:52:59 2023 +0200 6 files changed, 56 insertions(+), 20 deletions(-) create mode 100644 doc/contributing/maintaining-undici.md rename tools/{ => dep_updaters}/update-undici.sh (60%) ✔ Patches applied There are 2 commits in the PR. Attempting autorebase. Rebasing (2/4)https://github.com/nodejs/node/actions/runs/4665921327 |
Commit Queue failed- Loading data for nodejs/node/pull/47380 ✔ Done loading data for nodejs/node/pull/47380 ----------------------------------- PR info ------------------------------------ Title tools: move update-undici.sh to dep_updaters and create brotli maintaining md (#47380) Author Marco Ippolito (@marco-ippolito) Branch marco-ippolito:feat/improve-deps-update -> nodejs:main Labels meta, tools, author ready, commit-queue-rebase Commits 2 - doc: create maintaining-brotli.md - tools: move update-undici.sh to dep_updaters and create maintaining-u… Committers 1 - Marco Ippolito PR-URL: https://github.com/nodejs/node/pull/47380 Refs: https://github.com/nodejs/security-wg/issues/828 Reviewed-By: Yagiz Nizipli Reviewed-By: Paolo Insogna Reviewed-By: Rafael Gonzaga Reviewed-By: Luigi Pinca ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/47380 Refs: https://github.com/nodejs/security-wg/issues/828 Reviewed-By: Yagiz Nizipli Reviewed-By: Paolo Insogna Reviewed-By: Rafael Gonzaga Reviewed-By: Luigi Pinca -------------------------------------------------------------------------------- ℹ This PR was created on Mon, 03 Apr 2023 09:59:54 GMT ✔ Approvals: 4 ✔ - Yagiz Nizipli (@anonrig): https://github.com/nodejs/node/pull/47380#pullrequestreview-1369790275 ✔ - Paolo Insogna (@ShogunPanda): https://github.com/nodejs/node/pull/47380#pullrequestreview-1376060783 ✔ - Rafael Gonzaga (@RafaelGSS) (TSC): https://github.com/nodejs/node/pull/47380#pullrequestreview-1376541837 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/47380#pullrequestreview-1378029737 ✔ Last GitHub CI successful ℹ Green GitHub CI is sufficient -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/main up to date... From https://github.com/nodejs/node * branch main -> FETCH_HEAD ✔ origin/main is now up-to-date - Downloading patch for 47380 From https://github.com/nodejs/node * branch refs/pull/47380/merge -> FETCH_HEAD ✔ Fetched commits as 97d3912eb8dd..a41e4f114fee -------------------------------------------------------------------------------- [main cf836f3044] doc: create maintaining-brotli.md Author: Marco Ippolito Date: Mon Apr 3 10:50:40 2023 +0200 1 file changed, 26 insertions(+) create mode 100644 doc/contributing/maintaining-brotli.md Auto-merging .github/workflows/tools.yml [main 9533b8e3bf] tools: move update-undici.sh to dep_updaters and create maintaining-undici.md Author: Marco Ippolito Date: Mon Apr 3 11:52:59 2023 +0200 6 files changed, 56 insertions(+), 20 deletions(-) create mode 100644 doc/contributing/maintaining-undici.md rename tools/{ => dep_updaters}/update-undici.sh (60%) ✔ Patches applied There are 2 commits in the PR. Attempting autorebase. Rebasing (2/4)https://github.com/nodejs/node/actions/runs/4665993233 |
a41e4f1
to
af2fc9e
Compare
Commit Queue failed- Loading data for nodejs/node/pull/47380 ✔ Done loading data for nodejs/node/pull/47380 ----------------------------------- PR info ------------------------------------ Title tools: move update-undici.sh to dep_updaters and create brotli maintaining md (#47380) Author Marco Ippolito (@marco-ippolito) Branch marco-ippolito:feat/improve-deps-update -> nodejs:main Labels meta, tools, author ready, commit-queue-rebase Commits 2 - doc: create maintaining-brotli.md - tools: move update-undici.sh to dep_updaters and create maintain md Committers 1 - Marco Ippolito PR-URL: https://github.com/nodejs/node/pull/47380 Refs: https://github.com/nodejs/security-wg/issues/828 Reviewed-By: Yagiz Nizipli Reviewed-By: Paolo Insogna Reviewed-By: Rafael Gonzaga Reviewed-By: Luigi Pinca ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/47380 Refs: https://github.com/nodejs/security-wg/issues/828 Reviewed-By: Yagiz Nizipli Reviewed-By: Paolo Insogna Reviewed-By: Rafael Gonzaga Reviewed-By: Luigi Pinca -------------------------------------------------------------------------------- ⚠ Commits were pushed since the last review: ⚠ - tools: move update-undici.sh to dep_updaters and create maintain md ℹ This PR was created on Mon, 03 Apr 2023 09:59:54 GMT ✔ Approvals: 4 ✔ - Yagiz Nizipli (@anonrig): https://github.com/nodejs/node/pull/47380#pullrequestreview-1369790275 ✔ - Paolo Insogna (@ShogunPanda): https://github.com/nodejs/node/pull/47380#pullrequestreview-1376060783 ✔ - Rafael Gonzaga (@RafaelGSS) (TSC): https://github.com/nodejs/node/pull/47380#pullrequestreview-1376541837 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/47380#pullrequestreview-1378029737 ✔ Last GitHub CI successful ℹ Green GitHub CI is sufficient -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncuhttps://github.com/nodejs/node/actions/runs/4667850921 |
Landed in 25ad49b...4899998 |
PR-URL: #47380 Refs: nodejs/security-wg#828 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #47380 Refs: nodejs/security-wg#828 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #47380 Refs: nodejs/security-wg#828 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #47380 Refs: nodejs/security-wg#828 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #47380 Refs: nodejs/security-wg#828 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #47380 Refs: nodejs/security-wg#828 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: nodejs#47380 Refs: nodejs/security-wg#828 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: nodejs#47380 Refs: nodejs/security-wg#828 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Refs: nodejs/security-wg#828
@nodejs/undici
I'd also propose to create a folder under
contributing
that includes allmaintaining
files@nodejs/security-wg