-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Create or Update Pull Request Action
committed
Nov 19, 2023
1 parent
f6d288a
commit 0f98510
Showing
1 changed file
with
279 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,279 @@ | ||
| # Retrospective for `nodejs` from 2023-11-12 to 2023-11-19 | ||
|
|
||
| Reporting on 35 Issues from 26 authors, 65 Pull Requests from 38 authors, and 13 Discussions from 12 authors. | ||
|
|
||
|
|
||
| ### reliability | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - CI Reliability 2023-11-19 ([#723](https://github.com/nodejs/reliability/issues/723)) | ||
| - CI Reliability 2023-11-18 ([#722](https://github.com/nodejs/reliability/issues/722)) | ||
| - CI Reliability 2023-11-17 ([#721](https://github.com/nodejs/reliability/issues/721)) | ||
|
|
||
| ### node | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - `--env-file` does not support escaping quotes ([#50801](https://github.com/nodejs/node/issues/50801)) | ||
| - Imported modules should assume the type of the `.js` file matches the import condition type ([#50798](https://github.com/nodejs/node/issues/50798)) | ||
| - How to get worker.threadId inside a worker using worker_threads? ([#50797](https://github.com/nodejs/node/issues/50797)) | ||
| - document CLI `--input-type` default & limitations ([#50781](https://github.com/nodejs/node/issues/50781)) | ||
| - `assert.ok()` throwing `AssertionError` instead of provided `Error` object ([#50780](https://github.com/nodejs/node/issues/50780)) | ||
| - repl: /x/dg is autocompleted to bogus /x/dgram ([#50767](https://github.com/nodejs/node/issues/50767)) | ||
| - Nodejs 20.9 Internal Assertion Error ([#50765](https://github.com/nodejs/node/issues/50765)) | ||
| - Memory leak when piping between streams ([#50762](https://github.com/nodejs/node/issues/50762)) | ||
| - parallel/test-worker-nearheaplimit-deadlock is flaky ([#50761](https://github.com/nodejs/node/issues/50761)) | ||
| - ERR_INTERNAL_ASSERTION ([#50754](https://github.com/nodejs/node/issues/50754)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - tools: update eslint to 8.54.0 ([#50809](https://github.com/nodejs/node/pull/50809)) | ||
| - deps: update ngtcp2 to 1.0.1 ([#50808](https://github.com/nodejs/node/pull/50808)) | ||
| - tools: update lint-md-dependencies to [email protected] ([#50807](https://github.com/nodejs/node/pull/50807)) | ||
| - deps: update minimatch to 9.0.3 ([#50806](https://github.com/nodejs/node/pull/50806)) | ||
| - crypto: update root-certificates to 3.95 ([#50805](https://github.com/nodejs/node/pull/50805)) | ||
| - deps: update brotli to 1.1.0 ([#50804](https://github.com/nodejs/node/pull/50804)) | ||
| - deps: update zlib to 1.2.13.1-motley-5daffc7 ([#50803](https://github.com/nodejs/node/pull/50803)) | ||
| - deps: update c-ares to 1.22.0 ([#50800](https://github.com/nodejs/node/pull/50800)) | ||
| - doc: --input-type default & limitations ([#50796](https://github.com/nodejs/node/pull/50796)) | ||
| - test: replace forEach to for at test-webcrypto-sign-verify-ecdsa.js ([#50795](https://github.com/nodejs/node/pull/50795)) | ||
| - test: replace forEach test-http-server-multiheaders2 ([#50794](https://github.com/nodejs/node/pull/50794)) | ||
| - test: replace forEach with for-of in test-webcrypto-export-import-ec ([#50793](https://github.com/nodejs/node/pull/50793)) | ||
| - test: replace forEach with for-of in test-v8-serders.js ([#50791](https://github.com/nodejs/node/pull/50791)) | ||
| - test: replace foreach with for of ([#50790](https://github.com/nodejs/node/pull/50790)) | ||
| - test: replace forEach with for of in file test-trace-events-net.js ([#50789](https://github.com/nodejs/node/pull/50789)) | ||
| - test: replace forEach with for-of in test-url-relative ([#50788](https://github.com/nodejs/node/pull/50788)) | ||
| - test: replace foreach test-fs-readv.js test-fs-readv-sync.js ([#50787](https://github.com/nodejs/node/pull/50787)) | ||
| - test: replace foreach test-stream-pipe-unpipe-stream.js ([#50786](https://github.com/nodejs/node/pull/50786)) | ||
| - fix for loop ([#50785](https://github.com/nodejs/node/pull/50785)) | ||
| - test: replace forEach with for of in test-trace-events-api.js ([#50784](https://github.com/nodejs/node/pull/50784)) | ||
| - test: replace forEach() with for...of in test/parallel/test-util-log.js ([#50783](https://github.com/nodejs/node/pull/50783)) | ||
| - test: replace foreach with for of loop test-http-hostname-typechecking.js ([#50782](https://github.com/nodejs/node/pull/50782)) | ||
| - test: fix wpt tests for webstreams ([#50779](https://github.com/nodejs/node/pull/50779)) | ||
| - test: enable idlharness tests for encoding ([#50778](https://github.com/nodejs/node/pull/50778)) | ||
| - test_runner: add cobertura reporter ([#50777](https://github.com/nodejs/node/pull/50777)) | ||
| - inspector: use private fields instead of symbols ([#50776](https://github.com/nodejs/node/pull/50776)) | ||
| - lib: streamline process.binding() handling ([#50773](https://github.com/nodejs/node/pull/50773)) | ||
| - deps: update ada to 2.7.3 ([#50771](https://github.com/nodejs/node/pull/50771)) | ||
| - net: check pipe mode and path ([#50770](https://github.com/nodejs/node/pull/50770)) | ||
| - fs: fix to not return for void function ([#50769](https://github.com/nodejs/node/pull/50769)) | ||
| - lib: add experimental benchmark module ([#50768](https://github.com/nodejs/node/pull/50768)) | ||
| - benchmark: increase the iteration number to an appropriate value ([#50766](https://github.com/nodejs/node/pull/50766)) | ||
| - src: make ModifyCodeGenerationFromStrings more robust ([#50763](https://github.com/nodejs/node/pull/50763)) | ||
| - test: log more information in SEA tests ([#50759](https://github.com/nodejs/node/pull/50759)) | ||
| - lib,src,permissions: port path.resolve to C++ ([#50758](https://github.com/nodejs/node/pull/50758)) | ||
|
|
||
| #### Open Discussions | ||
|
|
||
| - Is there a plan to back port the V8 compilation cache fix to v18.x? ([#50799](https://github.com/orgs/nodejs/discussions/50799)) | ||
| - ERROR [unhandledRejection] read/write ECONNRESET ([#50148](https://github.com/orgs/nodejs/discussions/50148)) | ||
| - MaxListenersExceededWarning when installing or creating Node.js projects on macOS ([#49784](https://github.com/orgs/nodejs/discussions/49784)) | ||
| - Enabling FIPS results in `library has no ciphers` error ([#50543](https://github.com/orgs/nodejs/discussions/50543)) | ||
| - SSL routines::ca md too weak and workaround - ciphers: "DEFAULT:@SECLEVEL=0" isn't working ([#46545](https://github.com/orgs/nodejs/discussions/46545)) | ||
| - Unable to make tls1.0 connection via curl to app server running on node.js 18 ([#49634](https://github.com/orgs/nodejs/discussions/49634)) | ||
| - nodejs http module failing to parse chunk data and closes the connection ([#50755](https://github.com/orgs/nodejs/discussions/50755)) | ||
| - `fs.lstat` and `fs.readlink` not recognizing symbolic links in Windows ([#50722](https://github.com/orgs/nodejs/discussions/50722)) | ||
| - I just wanted to say that fs.rename is not suitable for file movement ([#50699](https://github.com/orgs/nodejs/discussions/50699)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - deps: update simdutf to 4.0.4 ([#50772](https://github.com/nodejs/node/pull/50772)) | ||
| - test: skip parallel/test-macos-app-sandbox if disk space < 120MB ([#50764](https://github.com/nodejs/node/pull/50764)) | ||
| - doc: fix fs.writeFileSync return value documentation ([#50760](https://github.com/nodejs/node/pull/50760)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - No test can be found on Windows using the wildcard paths ([#50757](https://github.com/nodejs/node/issues/50757)) | ||
|
|
||
| #### Closed Pull Requests | ||
|
|
||
| - deps: update timezone to 2023c ([#50810](https://github.com/nodejs/node/pull/50810)) | ||
| - test: change every forEach with for cicle ([#50792](https://github.com/nodejs/node/pull/50792)) | ||
| - Update trap-handler.h.patch ([#50775](https://github.com/nodejs/node/pull/50775)) | ||
| - Update trap-handler.h.patch ([#50774](https://github.com/nodejs/node/pull/50774)) | ||
| - deps: update simdutf to 4.0.4 ([#50756](https://github.com/nodejs/node/pull/50756)) | ||
|
|
||
| ### jenkins-alerts | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - test-ibm-ubuntu1804-x64-2 has low disk space ([#832](https://github.com/nodejs/jenkins-alerts/issues/832)) | ||
| - test-nearform_arm-win10_vs2019-arm64-1 is DOWN ([#828](https://github.com/nodejs/jenkins-alerts/issues/828)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - test-ibm-ubuntu1804-x64-2 has low disk space ([#831](https://github.com/nodejs/jenkins-alerts/issues/831)) | ||
| - test-ibm-ubuntu1804-x64-2 has low disk space ([#830](https://github.com/nodejs/jenkins-alerts/issues/830)) | ||
| - test-ibm-ubuntu1804-x64-2 has low disk space ([#829](https://github.com/nodejs/jenkins-alerts/issues/829)) | ||
| - test-ibm-ubuntu1804-x64-2 has low disk space ([#827](https://github.com/nodejs/jenkins-alerts/issues/827)) | ||
| - test-ibm-ubuntu1804-x64-2 has low disk space ([#826](https://github.com/nodejs/jenkins-alerts/issues/826)) | ||
|
|
||
| ### release-cloudflare-worker | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Enable dependabot for updating dependencies ([#79](https://github.com/nodejs/release-cloudflare-worker/issues/79)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - src: include error message when reporting failed R2 operations to sentry ([#82](https://github.com/nodejs/release-cloudflare-worker/pull/82)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - fix: don't treat file paths with 7z extension as directories ([#81](https://github.com/nodejs/release-cloudflare-worker/pull/81)) | ||
| - fix: retry R2 binding requests ([#80](https://github.com/nodejs/release-cloudflare-worker/pull/80)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - Error: head: We encountered an internal connectivity issue. Please try again. (10001) ([#78](https://github.com/nodejs/release-cloudflare-worker/issues/78)) | ||
|
|
||
| ### admin | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - 1password and vercel access ([#845](https://github.com/nodejs/admin/issues/845)) | ||
|
|
||
| ### performance | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - OutgoingMessage.writeFile ([#135](https://github.com/nodejs/performance/issues/135)) | ||
|
|
||
| ### node-core-utils | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - `git node vote` fails with `ERR_INVALID_ARG_TYPE` ([#753](https://github.com/nodejs/node-core-utils/issues/753)) | ||
|
|
||
| ### loaders | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Node.js Loaders Team Meeting 2023-11-21 ([#173](https://github.com/nodejs/loaders/issues/173)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - docs: include `--eval` support ([#174](https://github.com/nodejs/loaders/pull/174)) | ||
|
|
||
| ### nodejs.org | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Create `ChangelogModal` Component ([#6125](https://github.com/nodejs/nodejs.org/issues/6125)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - meta: updated next-intl and dependent packages ([#6130](https://github.com/nodejs/nodejs.org/pull/6130)) | ||
| - test: migrate jest -> node:test ([#6129](https://github.com/nodejs/nodejs.org/pull/6129)) | ||
| - feat: ChangelogModal Component ([#6126](https://github.com/nodejs/nodejs.org/pull/6126)) | ||
|
|
||
| #### Open Discussions | ||
|
|
||
| - What's next for Node.js Website? ([#5131](https://github.com/nodejs/nodejs.org/discussions/5131)) | ||
| - (event loop) ([#6106](https://github.com/nodejs/nodejs.org/discussions/6106)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - refactor: lazy-load sentry ([#6128](https://github.com/nodejs/nodejs.org/pull/6128)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - Consider refactoring the website？ ([#6127](https://github.com/nodejs/nodejs.org/issues/6127)) | ||
| - Error loading '@sentry/profiling-node' native module on Windows ([#6124](https://github.com/nodejs/nodejs.org/issues/6124)) | ||
|
|
||
| ### undici | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - HashWithOptions regex matches invalid attributes ([#2441](https://github.com/nodejs/undici/issues/2441)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - fix: `onHeaders` type declaration ([#2444](https://github.com/nodejs/undici/pull/2444)) | ||
|
|
||
| #### Open Discussions | ||
|
|
||
| - Optimizing thoughput to external calls ([#2439](https://github.com/nodejs/undici/discussions/2439)) | ||
| - OpenTelemetry tracing support? ([#1439](https://github.com/nodejs/undici/discussions/1439)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - fix: type comment position ([#2443](https://github.com/nodejs/undici/pull/2443)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - Node crash on second fetch call to caused by parseJSONFromBytes ([#2442](https://github.com/nodejs/undici/issues/2442)) | ||
|
|
||
| ### single-executable | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Add support monorepo ([#79](https://github.com/nodejs/single-executable/issues/79)) | ||
|
|
||
| ### help | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - getting error in npm run dev for react vite. ([#4297](https://github.com/nodejs/help/issues/4297)) | ||
|
|
||
| ### web-server-frameworks | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Cookies ([#93](https://github.com/nodejs/web-server-frameworks/issues/93)) | ||
|
|
||
| ### create-node-meeting-artifacts | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - Create meeting_base_package_metadata_interoperability_collab_space.md ([#144](https://github.com/nodejs/create-node-meeting-artifacts/pull/144)) | ||
|
|
||
| ### build | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - ansible: add loongson-clfs-loongarch64-1 ([#3572](https://github.com/nodejs/build/pull/3572)) | ||
|
|
||
| ### security-wg | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - build(deps): bump step-security/harden-runner from 2.6.0 to 2.6.1 ([#1153](https://github.com/nodejs/security-wg/pull/1153)) | ||
|
|
||
| ### node-addon-examples | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - Bump @types/node from 12.20.55 to 20.9.1 in /website ([#363](https://github.com/nodejs/node-addon-examples/pull/363)) | ||
| - Bump github/codeql-action from 2.20.3 to 2.22.7 ([#362](https://github.com/nodejs/node-addon-examples/pull/362)) | ||
| - Bump step-security/harden-runner from 2.4.1 to 2.6.1 ([#361](https://github.com/nodejs/node-addon-examples/pull/361)) | ||
|
|
||
| ### node-addon-api | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - build(deps): bump github/codeql-action from 2.22.5 to 2.22.7 ([#1413](https://github.com/nodejs/node-addon-api/pull/1413)) | ||
| - build(deps): bump step-security/harden-runner from 2.6.0 to 2.6.1 ([#1412](https://github.com/nodejs/node-addon-api/pull/1412)) | ||
|
|
||
| ### corepack | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - feat: update package manager versions ([#330](https://github.com/nodejs/corepack/pull/330)) | ||
|
|
||
| ### collaborators-public-votes | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - Node.js Mascot Voting ([#6](https://github.com/nodejs/collaborators-public-votes/pull/6)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - Vote for Node.js Mascot ([#5](https://github.com/nodejs/collaborators-public-votes/pull/5)) | ||
|
|
||
| ### Release | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - doc: add minutes for meeting 2023-11-16 ([#959](https://github.com/nodejs/Release/pull/959)) |