-
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
1b89833
commit b688ea1
Showing
1 changed file
with
233 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,233 @@ | ||
| # Retrospective for `nodejs` from 2024-10-20 to 2024-10-27 | ||
|
|
||
| Reporting on 55 Issues from 18 authors, 45 Pull Requests from 27 authors, and 7 Discussions from 7 authors. | ||
|
|
||
|
|
||
| ### reliability | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - CI Reliability 2024-10-27 ([#1009](https://github.com/nodejs/reliability/issues/1009)) | ||
| - CI Reliability 2024-10-26 ([#1008](https://github.com/nodejs/reliability/issues/1008)) | ||
| - CI Reliability 2024-10-25 ([#1007](https://github.com/nodejs/reliability/issues/1007)) | ||
|
|
||
| ### node | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - `fs.rmSync(path, { recursive: true, maxRetries: manyManyRetries})` throws on `ENFILE` ([#55555](https://github.com/nodejs/node/issues/55555)) | ||
| - structuredClone() uses wrong context in vm.runInContext() ([#55554](https://github.com/nodejs/node/issues/55554)) | ||
| - When using microtaskMode: 'afterEvaluate', the program gets stuck in the vm and cannot return to the main program. ([#55546](https://github.com/nodejs/node/issues/55546)) | ||
| - ERR_INTERNAL_ASSERTION from console.log(), TypeError: Converting circular structure to JSON ([#55539](https://github.com/nodejs/node/issues/55539)) | ||
| - file.path is only a getter ([#55538](https://github.com/nodejs/node/issues/55538)) | ||
| - Proposal: Deprecate `process.features.ipv6` and `process.features.uv` ([#55537](https://github.com/nodejs/node/issues/55537)) | ||
| - Invalid sqlite behavior about unused named parameter ([#55533](https://github.com/nodejs/node/issues/55533)) | ||
| - Setting a breakpoint in Chrome DevTools triggers fatal error in V8 when using top-level await ([#55532](https://github.com/nodejs/node/issues/55532)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - src: do not run IsWindowsBatchFile on non-windows ([#55560](https://github.com/nodejs/node/pull/55560)) | ||
| - deps: update nghttp2 to 1.64.0 ([#55559](https://github.com/nodejs/node/pull/55559)) | ||
| - deps: update acorn to 8.13.0 ([#55558](https://github.com/nodejs/node/pull/55558)) | ||
| - deps: update sqlite to 3.47.0 ([#55557](https://github.com/nodejs/node/pull/55557)) | ||
| - fs: fix parentPath returned from getDirent() when type is `UV_DIRENT_UNKNOWN` ([#55553](https://github.com/nodejs/node/pull/55553)) | ||
| - lib: make ALS default to AsyncContextFrame ([#55552](https://github.com/nodejs/node/pull/55552)) | ||
| - deps: update icu to 76.1 ([#55551](https://github.com/nodejs/node/pull/55551)) | ||
| - typings: fix `ModulesBinding` types ([#55549](https://github.com/nodejs/node/pull/55549)) | ||
| - fs: remove `dirent.path` ([#55548](https://github.com/nodejs/node/pull/55548)) | ||
| - fs: make `dirent.path` writable ([#55547](https://github.com/nodejs/node/pull/55547)) | ||
| - process: deprecate `features.{ipv6,uv}` and `features.tls_*` ([#55545](https://github.com/nodejs/node/pull/55545)) | ||
| - util: don't catch on circular toStringTag error ([#55544](https://github.com/nodejs/node/pull/55544)) | ||
| - module: simplify `findPackageJSON` implementation ([#55543](https://github.com/nodejs/node/pull/55543)) | ||
| - doc: move typescript support to active development ([#55536](https://github.com/nodejs/node/pull/55536)) | ||
| - doc: add suggested tsconfig for type stripping ([#55534](https://github.com/nodejs/node/pull/55534)) | ||
| - build: fix GN arg used in `generate_config_gypi.py` ([#55530](https://github.com/nodejs/node/pull/55530)) | ||
| - build: fix GN build for sqlite and nghttp2 ([#55529](https://github.com/nodejs/node/pull/55529)) | ||
| - Revert "fs,win: fix bug in paths with trailing slashes" ([#55527](https://github.com/nodejs/node/pull/55527)) | ||
| - src: support cancel for dns.lookup and dns.lookupService ([#55526](https://github.com/nodejs/node/pull/55526)) | ||
|
|
||
| #### Open Discussions | ||
|
|
||
| - Does supporting the Explicit Resource Management proposal hurt the ecosystem? ([#55342](https://github.com/orgs/nodejs/discussions/55342)) | ||
| - napi_create_external_buffer - what should happen when length is zero, but with a valid pointer? ([#55535](https://github.com/orgs/nodejs/discussions/55535)) | ||
| - Does Node expose a way of hooking into node:perf_hooks? ([#55528](https://github.com/orgs/nodejs/discussions/55528)) | ||
| - How to honor CRL if session resumption (using Session identifiers) is implemented? ([#55494](https://github.com/orgs/nodejs/discussions/55494)) | ||
| - require(esm) in v23 😯 ([#55475](https://github.com/orgs/nodejs/discussions/55475)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - Method toString broken for numbers in buffer ? ([#55556](https://github.com/nodejs/node/issues/55556)) | ||
| - .env files and Node 23 ([#55550](https://github.com/nodejs/node/issues/55550)) | ||
| - Add more specific assertions methods ([#55541](https://github.com/nodejs/node/issues/55541)) | ||
|
|
||
| #### Closed Pull Requests | ||
|
|
||
| - feat: add array method ([#55542](https://github.com/nodejs/node/pull/55542)) | ||
| - doc: add jazelly to collaborators ([#55531](https://github.com/nodejs/node/pull/55531)) | ||
|
|
||
| ### jenkins-alerts | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - test-equinix-ubuntu2204_container-armv7l-1 is DOWN ([#3021](https://github.com/nodejs/jenkins-alerts/issues/3021)) | ||
| - test-equinix-ubuntu2204_container-arm64-7 is DOWN ([#3020](https://github.com/nodejs/jenkins-alerts/issues/3020)) | ||
| - test-equinix-ubuntu2204_container-arm64-5 is DOWN ([#3019](https://github.com/nodejs/jenkins-alerts/issues/3019)) | ||
| - test-equinix-ubuntu2204_container-arm64-1 is DOWN ([#3017](https://github.com/nodejs/jenkins-alerts/issues/3017)) | ||
| - test-equinix-ubuntu2204_container-arm64-3 is DOWN ([#3018](https://github.com/nodejs/jenkins-alerts/issues/3018)) | ||
| - test-equinix-ubuntu1804_sharedlibs_container-arm64-3 is DOWN ([#3016](https://github.com/nodejs/jenkins-alerts/issues/3016)) | ||
| - test-equinix-ubuntu1804_sharedlibs_container-arm64-2 is DOWN ([#3015](https://github.com/nodejs/jenkins-alerts/issues/3015)) | ||
| - test-equinix-ubuntu1804_sharedlibs_container-arm64-1 is DOWN ([#3014](https://github.com/nodejs/jenkins-alerts/issues/3014)) | ||
| - test-equinix-ubuntu1804_container-arm64-1 is DOWN ([#3013](https://github.com/nodejs/jenkins-alerts/issues/3013)) | ||
| - test-equinix-rhel8_container-arm64-1 is DOWN ([#3012](https://github.com/nodejs/jenkins-alerts/issues/3012)) | ||
| - test-equinix-debian11_container-armv7l-1 is DOWN ([#3011](https://github.com/nodejs/jenkins-alerts/issues/3011)) | ||
|
|
||
| #### Closed Issues | ||
|
|
||
| - test-rackspace-win2022_vs2022-x64-1 has low disk space ([#3025](https://github.com/nodejs/jenkins-alerts/issues/3025)) | ||
| - test-digitalocean-ubuntu2204-x64-2 has low disk space ([#3024](https://github.com/nodejs/jenkins-alerts/issues/3024)) | ||
| - test-rackspace-win2022_vs2022-x64-1 has low disk space ([#3022](https://github.com/nodejs/jenkins-alerts/issues/3022)) | ||
| - test-softlayer-debian12-x64-1 has low disk space ([#3023](https://github.com/nodejs/jenkins-alerts/issues/3023)) | ||
| - test-softlayer-debian12-x64-1 has low disk space ([#3010](https://github.com/nodejs/jenkins-alerts/issues/3010)) | ||
| - test-rackspace-win2022_vs2022-x64-1 has low disk space ([#3009](https://github.com/nodejs/jenkins-alerts/issues/3009)) | ||
| - test-digitalocean-debian11-x64-1 has low disk space ([#3008](https://github.com/nodejs/jenkins-alerts/issues/3008)) | ||
| - test-softlayer-debian12-x64-1 has low disk space ([#3007](https://github.com/nodejs/jenkins-alerts/issues/3007)) | ||
| - test-digitalocean-ubuntu2204-x64-2 has low disk space ([#3006](https://github.com/nodejs/jenkins-alerts/issues/3006)) | ||
| - test-osuosl-rhel8-ppc64_le-4 has low disk space ([#3005](https://github.com/nodejs/jenkins-alerts/issues/3005)) | ||
| - test-softlayer-debian12-x64-1 has low disk space ([#3004](https://github.com/nodejs/jenkins-alerts/issues/3004)) | ||
| - test-ibm-rhel8-s390x-2 has low disk space ([#3003](https://github.com/nodejs/jenkins-alerts/issues/3003)) | ||
| - test-digitalocean-ubuntu2204_sharedlibs_container-x64-9 has low disk space ([#3002](https://github.com/nodejs/jenkins-alerts/issues/3002)) | ||
| - test-digitalocean-ubuntu2204_sharedlibs_container-x64-7 has low disk space ([#3001](https://github.com/nodejs/jenkins-alerts/issues/3001)) | ||
| - test-digitalocean-ubuntu2204_sharedlibs_container-x64-5 has low disk space ([#3000](https://github.com/nodejs/jenkins-alerts/issues/3000)) | ||
| - test-digitalocean-ubuntu2204_sharedlibs_container-x64-3 has low disk space ([#2999](https://github.com/nodejs/jenkins-alerts/issues/2999)) | ||
| - test-digitalocean-ubuntu2204_sharedlibs_container-x64-1 has low disk space ([#2998](https://github.com/nodejs/jenkins-alerts/issues/2998)) | ||
| - test-digitalocean-ubi81_container-x64-1 has low disk space ([#2997](https://github.com/nodejs/jenkins-alerts/issues/2997)) | ||
| - test-digitalocean-rhel8_arm_cross_container-x64-1 has low disk space ([#2995](https://github.com/nodejs/jenkins-alerts/issues/2995)) | ||
| - test-digitalocean-rhel9_arm_cross_container-x64-1 has low disk space ([#2996](https://github.com/nodejs/jenkins-alerts/issues/2996)) | ||
| - test-digitalocean-alpine318_container-x64-1 has low disk space ([#2994](https://github.com/nodejs/jenkins-alerts/issues/2994)) | ||
| - test-osuosl-rhel8-ppc64_le-4 has low disk space ([#2993](https://github.com/nodejs/jenkins-alerts/issues/2993)) | ||
| - test-softlayer-debian12-x64-1 has low disk space ([#2992](https://github.com/nodejs/jenkins-alerts/issues/2992)) | ||
| - test-digitalocean-ubuntu2204-x64-2 has low disk space ([#2991](https://github.com/nodejs/jenkins-alerts/issues/2991)) | ||
|
|
||
| ### performance | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Potential statup regression since on Node.js >= 22 ([#180](https://github.com/nodejs/performance/issues/180)) | ||
|
|
||
| ### admin | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - For visibility - enabled public packages for the org ([#929](https://github.com/nodejs/admin/issues/929)) | ||
|
|
||
| ### undici | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - RedirectHandler cleans headers too aggressively ([#3773](https://github.com/nodejs/undici/issues/3773)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - fix: assume blocking unless HEAD ([#3771](https://github.com/nodejs/undici/pull/3771)) | ||
| - fix: dns interceptor ip ttl ([#3770](https://github.com/nodejs/undici/pull/3770)) | ||
|
|
||
| #### Open Discussions | ||
|
|
||
| - Download file with redirect using unidici stream issue ([#3670](https://github.com/nodejs/undici/discussions/3670)) | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - add unsafe-url referrerPolicy test ([#3772](https://github.com/nodejs/undici/pull/3772)) | ||
| - fix http2 test ([#3769](https://github.com/nodejs/undici/pull/3769)) | ||
| - fix filename* parsing ([#3768](https://github.com/nodejs/undici/pull/3768)) | ||
|
|
||
| #### Closed Pull Requests | ||
|
|
||
| - Test queuing system bug ([#3774](https://github.com/nodejs/undici/pull/3774)) | ||
|
|
||
| ### nodejs.org | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - learn: make the links to the author profile clickable ([#7141](https://github.com/nodejs/nodejs.org/issues/7141)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - Refracto: update more code to ts ([#7148](https://github.com/nodejs/nodejs.org/pull/7148)) | ||
| - Docs: add .mjs example to WebAssembly usage section ([#7147](https://github.com/nodejs/nodejs.org/pull/7147)) | ||
| - Improve npm documentation for dependency management ([#7146](https://github.com/nodejs/nodejs.org/pull/7146)) | ||
| - feat: Introducing avatar tooltip ([#7143](https://github.com/nodejs/nodejs.org/pull/7143)) | ||
|
|
||
| #### Open Discussions | ||
|
|
||
| - change name of `next.something` and `next-data` ([#7145](https://github.com/nodejs/nodejs.org/discussions/7145)) | ||
|
|
||
| #### Closed Pull Requests | ||
|
|
||
| - new ([#7144](https://github.com/nodejs/nodejs.org/pull/7144)) | ||
| - adds link to github profile ([#7142](https://github.com/nodejs/nodejs.org/pull/7142)) | ||
|
|
||
| ### node-addon-api | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - ObjectWrap leak introduced in 8.2.0 ([#1602](https://github.com/nodejs/node-addon-api/issues/1602)) | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - chore: release v8.2.2 ([#1603](https://github.com/nodejs/node-addon-api/pull/1603)) | ||
|
|
||
| ### node-gyp | ||
|
|
||
| #### Open Issues | ||
|
|
||
| - Error building node-gyp 10.2.0 ([#3084](https://github.com/nodejs/node-gyp/issues/3084)) | ||
|
|
||
| ### TSC | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - doc: clarify regular member expectation ([#1641](https://github.com/nodejs/TSC/pull/1641)) | ||
|
|
||
| ### amaro | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - build: update to use common wasm builder ([#96](https://github.com/nodejs/amaro/pull/96)) | ||
|
|
||
| ### release-cloudflare-worker | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - docs: add sops ([#147](https://github.com/nodejs/release-cloudflare-worker/pull/147)) | ||
|
|
||
| ### wasm-builder | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - chore(main): release 0.0.9 ([#20](https://github.com/nodejs/wasm-builder/pull/20)) | ||
| - feat: update to using alpine 3.20 base container ([#19](https://github.com/nodejs/wasm-builder/pull/19)) | ||
| - chore(main): release 0.0.8 ([#18](https://github.com/nodejs/wasm-builder/pull/18)) | ||
| - Add the ability to run the release please action manually ([#17](https://github.com/nodejs/wasm-builder/pull/17)) | ||
|
|
||
| ### branch-diff | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - chore(deps): bump actions/setup-node from 4.0.4 to 4.1.0 ([#74](https://github.com/nodejs/branch-diff/pull/74)) | ||
|
|
||
| ### docker-node | ||
|
|
||
| #### Merged Pull Requests | ||
|
|
||
| - feat: Node.js 23.1.0 ([#2158](https://github.com/nodejs/docker-node/pull/2158)) | ||
|
|
||
| #### Closed Pull Requests | ||
|
|
||
| - feat: Node.js 23.1.0 ([#2157](https://github.com/nodejs/docker-node/pull/2157)) | ||
|
|
||
| ### node-api-headers | ||
|
|
||
| #### Open Pull Requests | ||
|
|
||
| - feat: update headers from nodejs/node tag v23.1.0 ([#54](https://github.com/nodejs/node-api-headers/pull/54)) |