From 3918d55b76daed27e7ee0489464aa5f9f146dd61 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tobias=20Nie=C3=9Fen?= Date: Sat, 19 Feb 2022 14:06:24 +0100 Subject: [PATCH 1/3] doc: add minutes for meeting 16 Feb 2022 Closes: https://github.com/nodejs/TSC/issues/1166 --- meetings/2022-02-16.md | 79 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 79 insertions(+) create mode 100644 meetings/2022-02-16.md diff --git a/meetings/2022-02-16.md b/meetings/2022-02-16.md new file mode 100644 index 00000000..8bf9d5a3 --- /dev/null +++ b/meetings/2022-02-16.md @@ -0,0 +1,79 @@ +# Node.js Technical Steering Committee (TSC) Meeting 2022-02-16 + +## Links + +* **Recording**: +* **GitHub Issue**: + +## Present + +* Antoine du Hamel @aduh95 (TSC) +* Matteo Collina @mcollina (TSC) +* Tobias Nießen @tniessen (TSC) +* Gireesh Punathil @gireeshpunathil (TSC) +* Darshan Sen @RaisinTen (TSC) +* Beth Griggs @BethGriggs (TSC) +* Сковорода Никита Андреевич @ChALkeR (TSC) +* Richard Lau @richardlau (TSC) +* Michaël Zasso @targos (TSC) + +## Agenda + +### Announcements + +* The Node.js copyright has been donated by Joyent/Samsung to the OpenJS Foundation +* OpenJS World CFP extended to 25th of February +* New collaborator! + +### CPC and Board Meeting Updates + +*Extracted from **tsc-agenda** labeled issues and pull requests from the **nodejs org** prior to the meeting. + +### nodejs/node + +* crypto: clarify `require("crypto").getRandomValues` is Node.js specific [#41782](https://github.com/nodejs/node/pull/41782) + * matteo: What is the problem? The question is unclear + * Antoine: James Snell added getRandomValue() as an alias to the require(‘crypto’) module. Tobias noted than there are might be webcompatibility. The compability issue is that getRandomValue() required this to be an instance of the spec Crypto object. + * It seems there are no conflict on the PR, just waiting for more approvals. + * Check out [#41779](https://github.com/nodejs/node/pull/41779), this includes the deprecation of getRandomValues(). Added the tsc-agenda label and we will talk about it next week. + +* Rename default branch from "master" to "main" [#33864](https://github.com/nodejs/node/issues/33864) + * Richard Lau: there are a lot of scripts that could break. + * Michael: it should not be possible to use git-node-land and land things to the wrong branch. + +### nodejs/TSC + +* Invite TSC members in the Google Calendar event for meetings [#1133](https://github.com/nodejs/TSC/issues/1133) + * Matteo: this is mostly done as everybody has received their invites. Removing the tsc-agenda label + +* vote: future of primordials [#1104](https://github.com/nodejs/TSC/issues/1104) + * Antoine: the vote has closed and the vote has been pasted in the PR. We can close the PR. + * Matteo: the PR needs to be finalized and landed. Antoine will do it. + * Antoine: do we want to do another vote? I’m currently busy right now. + * Matteo: If there are no other volunteers I propose we wait for another vote until the error path is finished. + * Michael: what does the vote mean? + * Antoine: the vote means that a PR that improves the tamper proofness of the error path can be blocked/reverted only if it causes a performance regression (or if there are other concerns regarding code quality unrelated to primordials) + +* Looking for feedback: Pointer compression in Node.js [#790](https://github.com/nodejs/TSC/issues/790) + * explains that we have a 4GB limit per process. + * Tobias: there are valid use for large memory usage + * Michael: what is the default memory limit for Node.js? + * Michael: buffers are not affected by pointer compression. + * Matteo: We agree we need a champion for this to progress it further. + +### nodejs/admin + +* Bot user for nodejs/corepack [#660](https://github.com/nodejs/admin/issues/660) + * Matteo to open an issue on nodejs/corepack to make dynamic downloads of the package manager version. + * +1 for having a bot to keep corepack updated in Node.js - maybe we can just update the eslint one to pull the new release. Antoine to do it. + +* blocklist import/export [#659](https://github.com/nodejs/admin/issues/659) + * On agenda for awareness. + +## Strategic Initiatives + +## Upcoming Meetings + +* **Node.js Foundation Calendar**: + +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. From 91343a11bd99cca3b37dd28cef4fd938ff24b7e4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tobias=20Nie=C3=9Fen?= Date: Sat, 19 Feb 2022 16:32:50 +0100 Subject: [PATCH 2/3] Update meetings/2022-02-16.md Co-authored-by: Richard Lau --- meetings/2022-02-16.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meetings/2022-02-16.md b/meetings/2022-02-16.md index 8bf9d5a3..8934ad63 100644 --- a/meetings/2022-02-16.md +++ b/meetings/2022-02-16.md @@ -21,7 +21,7 @@ ### Announcements -* The Node.js copyright has been donated by Joyent/Samsung to the OpenJS Foundation +* The Node.js trademarks have been donated by Joyent/Samsung to the OpenJS Foundation * OpenJS World CFP extended to 25th of February * New collaborator! From 33f00560ee06e970d6d2e06ce14b8cedb0d8e605 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tobias=20Nie=C3=9Fen?= Date: Sat, 19 Feb 2022 16:34:00 +0100 Subject: [PATCH 3/3] Update meetings/2022-02-16.md --- meetings/2022-02-16.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meetings/2022-02-16.md b/meetings/2022-02-16.md index 8934ad63..2b2f314b 100644 --- a/meetings/2022-02-16.md +++ b/meetings/2022-02-16.md @@ -32,7 +32,7 @@ ### nodejs/node * crypto: clarify `require("crypto").getRandomValues` is Node.js specific [#41782](https://github.com/nodejs/node/pull/41782) - * matteo: What is the problem? The question is unclear + * Matteo: What is the problem? The question is unclear * Antoine: James Snell added getRandomValue() as an alias to the require(‘crypto’) module. Tobias noted than there are might be webcompatibility. The compability issue is that getRandomValue() required this to be an instance of the spec Crypto object. * It seems there are no conflict on the PR, just waiting for more approvals. * Check out [#41779](https://github.com/nodejs/node/pull/41779), this includes the deprecation of getRandomValues(). Added the tsc-agenda label and we will talk about it next week.