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#47380marco-ippolito wants to merge 2 commits intonodejs:mainfrom
Conversation
|
Review requested:
|
f89a376 to
3fb789c
Compare
aduh95
left a comment
There was a problem hiding this comment.
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
RafaelGSS
left a comment
There was a problem hiding this comment.
The title says maintaining undici, but you are adding maintaining-brotli as well. Could you please adjust it?
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
contributingthat includes allmaintainingfiles@nodejs/security-wg