-
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.
Co-authored-by: Create or Update Pull Request Action <[email protected]>
- Loading branch information
1 parent
87fc8df
commit 5fa3829
Showing
1 changed file
with
206 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,206 @@ | ||
| # Retrospective for `nodejs` from 2024-09-08 to 2024-09-15 | ||
|
|
||
| Reporting on 36 Issues from 22 authors, 64 Pull Requests from 33 authors, and 9 Discussions from 9 authors. | ||
|
|
||
|
|
||
| ### help | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Subtests cause the parent `test` to stop showing up as green even if it asserts. ([#4482](https://github.com/nodejs/help/issues/4482)) | ||
| - How to build nodejs with -fPIC? ([#4481](https://github.com/nodejs/help/issues/4481)) | ||
|
|
||
| ### reliability | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - CI Reliability 2024-09-15 ([#967](https://github.com/nodejs/reliability/issues/967)) | ||
| - CI Reliability 2024-09-14 ([#966](https://github.com/nodejs/reliability/issues/966)) | ||
| - CI Reliability 2024-09-13 ([#965](https://github.com/nodejs/reliability/issues/965)) | ||
|
|
||
| ### node | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - [stream] uncatchable error thrown during piping if source and dest don't have same objectMode setting ([#54945](https://github.com/nodejs/node/issues/54945)) | ||
| - Data URLs can't have query params ([#54944](https://github.com/nodejs/node/issues/54944)) | ||
| - Typeless Module Detection does not report unsettled top-level await ([#54931](https://github.com/nodejs/node/issues/54931)) | ||
| - Support clearing Node.js cache for package.json ([#54924](https://github.com/nodejs/node/issues/54924)) | ||
| - Error Compiling Node.js v20.2.0 for Android (x86_64) using android-ndk-r25c ([#54919](https://github.com/nodejs/node/issues/54919)) | ||
| - Deadlock at process shutdown ([#54918](https://github.com/nodejs/node/issues/54918)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - test: wait for stream finish when --test-force-exit ([#54953](https://github.com/nodejs/node/pull/54953)) | ||
| - deps: patch V8 to 12.8.374.33 ([#54952](https://github.com/nodejs/node/pull/54952)) | ||
| - tools: update github_reporter to 1.7.1 ([#54951](https://github.com/nodejs/node/pull/54951)) | ||
| - deps: update acorn-walk to 8.3.4 ([#54950](https://github.com/nodejs/node/pull/54950)) | ||
| - cli: ensure --run has proper pwd ([#54949](https://github.com/nodejs/node/pull/54949)) | ||
| - test: remove `getCallSite` from common ([#54947](https://github.com/nodejs/node/pull/54947)) | ||
| - lib: prefer iterable weak set than WeakRefs in AbortSignal.any ([#54943](https://github.com/nodejs/node/pull/54943)) | ||
| - module: remove duplicated import ([#54942](https://github.com/nodejs/node/pull/54942)) | ||
| - src: remove duplicate code setting AF_INET ([#54939](https://github.com/nodejs/node/pull/54939)) | ||
| - meta: fix stalled issue bot ([#54936](https://github.com/nodejs/node/pull/54936)) | ||
| - test: fix invalid test ([#54935](https://github.com/nodejs/node/pull/54935)) | ||
| - src: use v8::LocalVector instead of MaybeStackBuffer<Value> ([#54934](https://github.com/nodejs/node/pull/54934)) | ||
| - module: support loading entrypoint as url ([#54933](https://github.com/nodejs/node/pull/54933)) | ||
| - test: adding more tests for strip-types ([#54929](https://github.com/nodejs/node/pull/54929)) | ||
| - benchmark: --no-warnings to avoid DEP/ExpWarn log ([#54928](https://github.com/nodejs/node/pull/54928)) | ||
| - lib: add flag to drop connection when running in cluster mode ([#54927](https://github.com/nodejs/node/pull/54927)) | ||
| - test: adjust test-tls-junk-server for OpenSSL32 ([#54926](https://github.com/nodejs/node/pull/54926)) | ||
| - test,crypto: update WebCryptoAPI WPT ([#54925](https://github.com/nodejs/node/pull/54925)) | ||
| - doc: fix broken Android building link ([#54922](https://github.com/nodejs/node/pull/54922)) | ||
| - test: remove duplicate skip AIX ([#54917](https://github.com/nodejs/node/pull/54917)) | ||
| - test: merge knownGlobals in common ([#54915](https://github.com/nodejs/node/pull/54915)) | ||
| - src: parse dotenv with the rest of the options ([#54913](https://github.com/nodejs/node/pull/54913)) | ||
| - src: move evp stuff to ncrypto ([#54911](https://github.com/nodejs/node/pull/54911)) | ||
| - build: describe the fix when using ndk to compile android node.js on x86_64 host ([#54908](https://github.com/nodejs/node/pull/54908)) | ||
| - fs: translate error code properly in cpSync ([#54906](https://github.com/nodejs/node/pull/54906)) | ||
| - benchmark: create benchmark for typescript ([#54904](https://github.com/nodejs/node/pull/54904)) | ||
|
|
||
| #### Open Discussions | ||
|
|
||
| - now that --experimental-network-imports has been removed, is there a work around to import files from a server? ([#54948](https://github.com/orgs/nodejs/discussions/54948)) | ||
| - Import module from global NPM packages if it cannot find module in local package ([#54916](https://github.com/orgs/nodejs/discussions/54916)) | ||
| - How to handle asynchronous operations with `async`/`await`? ([#54932](https://github.com/orgs/nodejs/discussions/54932)) | ||
| - UNABLE_TO_GET_ISSUER_CERT when using self signed certificate inside docker image ([#52045](https://github.com/orgs/nodejs/discussions/52045)) | ||
| - The future of NodeJS ([#54700](https://github.com/orgs/nodejs/discussions/54700)) | ||
| - Understanding mock.restoreAll() and mock.reset() in Node.js Test Runner ([#54887](https://github.com/orgs/nodejs/discussions/54887)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - test: adjust tls test for OpenSSL32 ([#54909](https://github.com/nodejs/node/pull/54909)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - Worker file extension bug ([#54941](https://github.com/nodejs/node/issues/54941)) | ||
| - Telegram ([#54940](https://github.com/nodejs/node/issues/54940)) | ||
| - [v22.8.0] - DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead. ([#54938](https://github.com/nodejs/node/issues/54938)) | ||
| - Bug related to using `Object.__proto__` within async call? ([#54930](https://github.com/nodejs/node/issues/54930)) | ||
| - ERR_INTERNAL_ASSERTION when loading ESM from CJS after compiling with `tsc` ([#54923](https://github.com/nodejs/node/issues/54923)) | ||
| - EFAULT buf outside accessible address space in a write system call on Raspberry Pi 4 (running 64bit bookworm) ([#54921](https://github.com/nodejs/node/issues/54921)) | ||
| - Draft Bot ([#54914](https://github.com/nodejs/node/issues/54914)) | ||
| - Error while running "npm run dev" ([#54910](https://github.com/nodejs/node/issues/54910)) | ||
| - loadEnvFile doesn't ignore comments ([#54907](https://github.com/nodejs/node/issues/54907)) | ||
| - Allocation failed - JavaScript heap out of memory ([#54905](https://github.com/nodejs/node/issues/54905)) | ||
|
|
||
| #### Closed Pull Requests | ||
|
|
||
| - Nodejs2024 ([#54946](https://github.com/nodejs/node/pull/54946)) | ||
| - Improved code ([#54937](https://github.com/nodejs/node/pull/54937)) | ||
| - Update README.md ([#54920](https://github.com/nodejs/node/pull/54920)) | ||
| - better ([#54912](https://github.com/nodejs/node/pull/54912)) | ||
|
|
||
| ### jenkins-alerts | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - test-loongson-clfs_loong64-1 is DOWN ([#2870](https://github.com/nodejs/jenkins-alerts/issues/2870)) | ||
| - test-rackspace-win2022_vs2022-x64-3 has low disk space ([#2869](https://github.com/nodejs/jenkins-alerts/issues/2869)) | ||
| - test-rackspace-win2022_vs2022-x64-3 has low disk space ([#2868](https://github.com/nodejs/jenkins-alerts/issues/2868)) | ||
| - test-ibm-aix72-ppc64_be-1 is DOWN ([#2867](https://github.com/nodejs/jenkins-alerts/issues/2867)) | ||
| - test-mnx-ubuntu2204-x64-1 has low disk space ([#2866](https://github.com/nodejs/jenkins-alerts/issues/2866)) | ||
| - test-ibm-ubuntu2204-x64-1 has low disk space ([#2865](https://github.com/nodejs/jenkins-alerts/issues/2865)) | ||
| - test-mnx-ubuntu2204-x64-1 has low disk space ([#2864](https://github.com/nodejs/jenkins-alerts/issues/2864)) | ||
| - test-loongson-clfs_loong64-1 is DOWN ([#2863](https://github.com/nodejs/jenkins-alerts/issues/2863)) | ||
|
|
||
| ### nodejs.org | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Is orama search on the main page broken? ([#7042](https://github.com/nodejs/nodejs.org/issues/7042)) | ||
| - Redesign the Downloads page tabs to reflect on latest agreements with Package Maintenance WG ([#7037](https://github.com/nodejs/nodejs.org/issues/7037)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - Package: `compile-mdx` ([#7044](https://github.com/nodejs/nodejs.org/pull/7044)) | ||
| - fix(action): correct lint path, clean up warnings, fix lints ([#7043](https://github.com/nodejs/nodejs.org/pull/7043)) | ||
| - [automated]: crowdin sync ([#7040](https://github.com/nodejs/nodejs.org/pull/7040)) | ||
| - Fix dropdown cutoff issue for long text in mobile view (#7035) ([#7039](https://github.com/nodejs/nodejs.org/pull/7039)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - fix: added missing cache ([#7041](https://github.com/nodejs/nodejs.org/pull/7041)) | ||
| - chore: update dep ([#7038](https://github.com/nodejs/nodejs.org/pull/7038)) | ||
|
|
||
| ### undici | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - EFAULT: bad address in system call argument, write ([#3601](https://github.com/nodejs/undici/issues/3601)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - fetch: make consumeBody sync ([#3606](https://github.com/nodejs/undici/pull/3606)) | ||
| - fetch: avoid async function in mainFetch to generate response ([#3605](https://github.com/nodejs/undici/pull/3605)) | ||
| - fetch: pullAlgorithm passes the async resume function through ([#3604](https://github.com/nodejs/undici/pull/3604)) | ||
| - fetch: make fullyReadBody sync ([#3603](https://github.com/nodejs/undici/pull/3603)) | ||
| - feat: dispatcher _buildConnector() method ([#3602](https://github.com/nodejs/undici/pull/3602)) | ||
| - jsdoc: add jsdoc to lib/web/fetch/constants.js ([#3597](https://github.com/nodejs/undici/pull/3597)) | ||
| - fix: throw on retry when payload is consume by downstream ([#3596](https://github.com/nodejs/undici/pull/3596)) | ||
| - feat(Backport v6.x): move throwOnError to interceptor ([#3595](https://github.com/nodejs/undici/pull/3595)) | ||
| - Fix borken link in readme ([#3591](https://github.com/nodejs/undici/pull/3591)) | ||
|
|
||
| #### Open Discussions | ||
|
|
||
| - Backporting of PRs ([#3526](https://github.com/nodejs/undici/discussions/3526)) | ||
| - How do we manage releases and pre-releases ([#3590](https://github.com/nodejs/undici/discussions/3590)) | ||
| - Does undici take care of consuming the response body if throwOnError: true? ([#3440](https://github.com/nodejs/undici/discussions/3440)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - Update WPT ([#3607](https://github.com/nodejs/undici/pull/3607)) | ||
| - diagnostics-channel: use not deprecated subscribe fn ([#3600](https://github.com/nodejs/undici/pull/3600)) | ||
| - chore: remove unused `pre-commit` dependency ([#3599](https://github.com/nodejs/undici/pull/3599)) | ||
| - Update WPT ([#3598](https://github.com/nodejs/undici/pull/3598)) | ||
| - docs(Backport v6.x): Fix signature of RetryHandler ([#3594](https://github.com/nodejs/undici/pull/3594)) | ||
| - fix: husky deprecation warning ([#3593](https://github.com/nodejs/undici/pull/3593)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - husky deprecation warning when trying to add a commit ([#3592](https://github.com/nodejs/undici/issues/3592)) | ||
|
|
||
| #### Closed Pull Requests | ||
|
|
||
| - fetch: optimize .arrayBuffer() mixin ([#3609](https://github.com/nodejs/undici/pull/3609)) | ||
| - fetch: optimize .bytes() mixin ([#3608](https://github.com/nodejs/undici/pull/3608)) | ||
|
|
||
| ### build | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Optimize disk usage by deleting successful build logs after 7 days. ([#3904](https://github.com/nodejs/build/issues/3904)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - ansible: install gcc12 on IBM i ([#3907](https://github.com/nodejs/build/pull/3907)) | ||
| - Orka template updates ([#3906](https://github.com/nodejs/build/pull/3906)) | ||
| - feat: adjust compresser script to work with print0 and paths ([#3905](https://github.com/nodejs/build/pull/3905)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - ansible: repurpose test-ibm-ubuntu2204-x64-2 ([#3909](https://github.com/nodejs/build/pull/3909)) | ||
| - jenkins: select gcc12 for Node 23 on IBM i ([#3908](https://github.com/nodejs/build/pull/3908)) | ||
| - ansible: replace Rackspace ubuntu1604-x64 machines ([#3903](https://github.com/nodejs/build/pull/3903)) | ||
| - feat: Add log compression script to reduce jenkins disk size ([#3902](https://github.com/nodejs/build/pull/3902)) | ||
|
|
||
| ### TSC | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Node.js Maintainers Threat Model: Access per Group table ([#1618](https://github.com/nodejs/TSC/issues/1618)) | ||
|
|
||
| ### security-wg | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - doc: add 2024-09-12 meeting notes ([#1380](https://github.com/nodejs/security-wg/pull/1380)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - update core index.json ([#1379](https://github.com/nodejs/security-wg/pull/1379)) | ||
| - OpenSSF Scorecard Report Updated ([#1378](https://github.com/nodejs/security-wg/pull/1378)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - OpenSSF Scorecard Report Updated! ([#1377](https://github.com/nodejs/security-wg/issues/1377)) |