-
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 <create-or-update-pull-request@users.noreply.github.com>
- Loading branch information
1 parent
5fa3829
commit 6b9737a
Showing
1 changed file
with
234 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,234 @@ | ||
# Retrospective for `nodejs` from 2024-09-15 to 2024-09-22 | ||
|
||
Reporting on 30 Issues from 23 authors, 70 Pull Requests from 26 authors, and 10 Discussions from 9 authors. | ||
|
||
|
||
### reliability | ||
|
||
#### Open Issues | ||
|
||
- CI Reliability 2024-09-22 ([#974](https://github.com/nodejs/reliability/issues/974)) | ||
- CI Reliability 2024-09-21 ([#973](https://github.com/nodejs/reliability/issues/973)) | ||
- CI Reliability 2024-09-20 ([#972](https://github.com/nodejs/reliability/issues/972)) | ||
- CI Reliability 2024-09-19 ([#971](https://github.com/nodejs/reliability/issues/971)) | ||
|
||
### node | ||
|
||
#### Open Issues | ||
|
||
- Could not report code coverage for file in sourcemap ([#55054](https://github.com/nodejs/node/issues/55054)) | ||
- Nominating @pmarchini to be a collaborator ([#55053](https://github.com/nodejs/node/issues/55053)) | ||
- Exception during run (mocha) ([#55049](https://github.com/nodejs/node/issues/55049)) | ||
- pipe() allow destroying source! ([#55010](https://github.com/nodejs/node/issues/55010)) | ||
- Any idea when Single executable application feature will reach stable status? ([#55007](https://github.com/nodejs/node/issues/55007)) | ||
- Support `net.BlockList` in `http.Agent` options ([#55000](https://github.com/nodejs/node/issues/55000)) | ||
- http2 server ECONNRESET on http1.1 request ([#54998](https://github.com/nodejs/node/issues/54998)) | ||
|
||
#### Open Pull Requests | ||
|
||
- deps: patch V8 to 12.9.202.19 ([#55057](https://github.com/nodejs/node/pull/55057)) | ||
- deps: update timezone to 2024b ([#55056](https://github.com/nodejs/node/pull/55056)) | ||
- test_runner: throw on invalid source map ([#55055](https://github.com/nodejs/node/pull/55055)) | ||
- src: add and use MaybeStackBuffer v8::LocalVector ([#55052](https://github.com/nodejs/node/pull/55052)) | ||
- test: execute shell directly for refresh() ([#55051](https://github.com/nodejs/node/pull/55051)) | ||
- lib: remove lib/internal/idna.js ([#55050](https://github.com/nodejs/node/pull/55050)) | ||
- stream: construct correct prototype to transferred streams ([#55047](https://github.com/nodejs/node/pull/55047)) | ||
- lib: prefer optional chaining ([#55045](https://github.com/nodejs/node/pull/55045)) | ||
- lib: prefer logical assignment ([#55044](https://github.com/nodejs/node/pull/55044)) | ||
- doc: correct `cleanup` option in stream.(promises.)finished ([#55043](https://github.com/nodejs/node/pull/55043)) | ||
- tools: unlock versions of irrelevant DB deps ([#55042](https://github.com/nodejs/node/pull/55042)) | ||
- crypto: ensure invalid SubtleCrypto JWK data import results in DataError ([#55041](https://github.com/nodejs/node/pull/55041)) | ||
- benchmark: add webstorage benchmark ([#55040](https://github.com/nodejs/node/pull/55040)) | ||
- test_runner: require `--enable-source-maps` for source map coverage ([#55039](https://github.com/nodejs/node/pull/55039)) | ||
- test_runner: report error on missing sourcemap source ([#55037](https://github.com/nodejs/node/pull/55037)) | ||
- test: make `test-runner-assert` more robust ([#55036](https://github.com/nodejs/node/pull/55036)) | ||
- process: remove `process.assert` ([#55035](https://github.com/nodejs/node/pull/55035)) | ||
- fs: check subdir correctly in cpSync ([#55033](https://github.com/nodejs/node/pull/55033)) | ||
- meta: re-enable PR stalebot ([#55032](https://github.com/nodejs/node/pull/55032)) | ||
- test: update tls test to support OpenSSL32 ([#55030](https://github.com/nodejs/node/pull/55030)) | ||
- test,crypto: update WebCryptoAPI WPT ([#55029](https://github.com/nodejs/node/pull/55029)) | ||
- test: do not assume `process.execPath` contains no spaces ([#55028](https://github.com/nodejs/node/pull/55028)) | ||
- benchmark: use options.filename when, for esm, require.main.filename is undefined ([#55027](https://github.com/nodejs/node/pull/55027)) | ||
- test: fix `test-vm-context-dont-contextify` when path contains a space ([#55026](https://github.com/nodejs/node/pull/55026)) | ||
- build: display free disk space after build in the test-macOS workflow ([#55025](https://github.com/nodejs/node/pull/55025)) | ||
- src: decode native error messages as UTF-8 ([#55024](https://github.com/nodejs/node/pull/55024)) | ||
- doc: add missing EventSource docs to globals ([#55022](https://github.com/nodejs/node/pull/55022)) | ||
- zlib: set zlib.bytesRead end-of-life ([#55020](https://github.com/nodejs/node/pull/55020)) | ||
- crypto: runtime deprecate crypto.fips ([#55019](https://github.com/nodejs/node/pull/55019)) | ||
- util: runtime deprecate toUSVString() ([#55018](https://github.com/nodejs/node/pull/55018)) | ||
- url: runtime deprecate url.parse ([#55017](https://github.com/nodejs/node/pull/55017)) | ||
- deps: update V8 to 13.0 ([#55014](https://github.com/nodejs/node/pull/55014)) | ||
- test: remove interval and give more time to unsync ([#55006](https://github.com/nodejs/node/pull/55006)) | ||
- test: deflake test-http-remove-header-stays-removed ([#55004](https://github.com/nodejs/node/pull/55004)) | ||
- esm: do not interpret `"main"` as a URL ([#55003](https://github.com/nodejs/node/pull/55003)) | ||
|
||
#### Open Discussions | ||
|
||
- How to solve this error npm ERR! code EEXIST? ([#55002](https://github.com/orgs/nodejs/discussions/55002)) | ||
- Warning potential scam emails ([#53726](https://github.com/orgs/nodejs/discussions/53726)) | ||
- Import module from global NPM packages if it cannot find module in local package ([#54916](https://github.com/orgs/nodejs/discussions/54916)) | ||
- try to build an extended version of node to log every undefined property visiting ([#46960](https://github.com/orgs/nodejs/discussions/46960)) | ||
- Understanding mock.restoreAll() and mock.reset() in Node.js Test Runner ([#54887](https://github.com/orgs/nodejs/discussions/54887)) | ||
- 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)) | ||
|
||
#### Merged Pull Requests | ||
|
||
- test: fix `soucre` to `source` ([#55038](https://github.com/nodejs/node/pull/55038)) | ||
- doc: cover --experimental-test-module-mocks flag ([#55021](https://github.com/nodejs/node/pull/55021)) | ||
- test: adjust tls-set-ciphers for OpenSSL32 ([#55016](https://github.com/nodejs/node/pull/55016)) | ||
- src: revert filesystem::path changes ([#55015](https://github.com/nodejs/node/pull/55015)) | ||
- test_runner: replace ansi clear with ansi reset ([#55013](https://github.com/nodejs/node/pull/55013)) | ||
- build: fix eslint makefile target ([#54999](https://github.com/nodejs/node/pull/54999)) | ||
|
||
#### Closed Issues | ||
|
||
- Deprecation Warning After Node Update ([#55046](https://github.com/nodejs/node/issues/55046)) | ||
- Can not build on macOS 14.6.1 with clang 16 ([#55023](https://github.com/nodejs/node/issues/55023)) | ||
- Please add native support of webrtc in node. ([#55012](https://github.com/nodejs/node/issues/55012)) | ||
- JavaScript error ([#55011](https://github.com/nodejs/node/issues/55011)) | ||
- `dot` reporter causes the terminal to clear since Node v22.8.0 ([#55009](https://github.com/nodejs/node/issues/55009)) | ||
- [ invalid ] ([#55008](https://github.com/nodejs/node/issues/55008)) | ||
- HTTPS CRL Usage not working ([#55005](https://github.com/nodejs/node/issues/55005)) | ||
|
||
#### Closed Pull Requests | ||
|
||
- test_runner: don't set coverage threshold exitCode until `test:summary` ([#55048](https://github.com/nodejs/node/pull/55048)) | ||
- domain: EoL deprecate `MakeCallback` with `domain` property ([#55034](https://github.com/nodejs/node/pull/55034)) | ||
- added test file ([#55031](https://github.com/nodejs/node/pull/55031)) | ||
- test: add asserts to validate test assumptions ([#54997](https://github.com/nodejs/node/pull/54997)) | ||
|
||
### web-server-frameworks | ||
|
||
#### Open Issues | ||
|
||
- Node.js Web Server Frameworks Meeting 2024-09-24 ([#124](https://github.com/nodejs/web-server-frameworks/issues/124)) | ||
|
||
### loaders | ||
|
||
#### Open Issues | ||
|
||
- Node.js Loaders Team Meeting 2024-09-24 ([#227](https://github.com/nodejs/loaders/issues/227)) | ||
|
||
### build | ||
|
||
#### Open Pull Requests | ||
|
||
- promote: add expected assets for Node.js v23 ([#3915](https://github.com/nodejs/build/pull/3915)) | ||
|
||
#### Closed Issues | ||
|
||
- FreeBSD builds timing out ([#3914](https://github.com/nodejs/build/issues/3914)) | ||
|
||
### undici | ||
|
||
#### Open Pull Requests | ||
|
||
- feat: add new hooks api ([#3636](https://github.com/nodejs/undici/pull/3636)) | ||
- fix(websocket): use binary operations instead of a Set for closeStatus ([#3635](https://github.com/nodejs/undici/pull/3635)) | ||
- Update WPT ([#3634](https://github.com/nodejs/undici/pull/3634)) | ||
- Remove symbols from web specs ([#3633](https://github.com/nodejs/undici/pull/3633)) | ||
- build(deps): bump node from `3cb4748` to `83b4d7b` in /build ([#3621](https://github.com/nodejs/undici/pull/3621)) | ||
|
||
#### Open Discussions | ||
|
||
- What's the best way to reset the MockPool between tests? ([#3619](https://github.com/nodejs/undici/discussions/3619)) | ||
- HTTP/2 client inside HTTP/1.1 proxy connection ([#3613](https://github.com/nodejs/undici/discussions/3613)) | ||
- Does undici take care of consuming the response body if throwOnError: true? ([#3440](https://github.com/nodejs/undici/discussions/3440)) | ||
- Backporting of PRs ([#3526](https://github.com/nodejs/undici/discussions/3526)) | ||
|
||
#### Merged Pull Requests | ||
|
||
- handle body errors ([#3632](https://github.com/nodejs/undici/pull/3632)) | ||
- make cloned request inherit dispatcher ([#3631](https://github.com/nodejs/undici/pull/3631)) | ||
- fix: fire close on failed WebSocket connection ([#3628](https://github.com/nodejs/undici/pull/3628)) | ||
- Update WPT ([#3626](https://github.com/nodejs/undici/pull/3626)) | ||
- append crlf to formdata body ([#3625](https://github.com/nodejs/undici/pull/3625)) | ||
- Update WPT ([#3622](https://github.com/nodejs/undici/pull/3622)) | ||
|
||
#### Closed Issues | ||
|
||
- undici.Request.clone does not copy dispatcher ([#3630](https://github.com/nodejs/undici/issues/3630)) | ||
- 🏁 RoSctober 2024: The Open Source Developer Challenge is about to start! 🎉 ([#3623](https://github.com/nodejs/undici/issues/3623)) | ||
- When using fetch for FormData upload files, lack of"\r\n" ([#3624](https://github.com/nodejs/undici/issues/3624)) | ||
|
||
#### Closed Pull Requests | ||
|
||
- feat: new hooks ([#3629](https://github.com/nodejs/undici/pull/3629)) | ||
- Revert "fix: fire `close` on failed WebSocket connection" ([#3627](https://github.com/nodejs/undici/pull/3627)) | ||
- fix: dont crash on invalid brotli payload ([#3620](https://github.com/nodejs/undici/pull/3620)) | ||
|
||
### jenkins-alerts | ||
|
||
#### Closed Issues | ||
|
||
- test-rackspace-win2022_vs2022-x64-4 has low disk space ([#2877](https://github.com/nodejs/jenkins-alerts/issues/2877)) | ||
|
||
### nodejs-dist-indexer | ||
|
||
#### Open Issues | ||
|
||
- Test & Maybe Release workflow fails ([#32](https://github.com/nodejs/nodejs-dist-indexer/issues/32)) | ||
|
||
#### Merged Pull Requests | ||
|
||
- chore(deps): bump actions/setup-node from 4.0.3 to 4.0.4 ([#31](https://github.com/nodejs/nodejs-dist-indexer/pull/31)) | ||
|
||
### Release | ||
|
||
#### Open Issues | ||
|
||
- @nodejs/releasers health ([#1036](https://github.com/nodejs/Release/issues/1036)) | ||
- Node.js 23 release plan ([#1034](https://github.com/nodejs/Release/issues/1034)) | ||
|
||
#### Open Pull Requests | ||
|
||
- chore: remove lts team ([#1037](https://github.com/nodejs/Release/pull/1037)) | ||
|
||
#### Merged Pull Requests | ||
|
||
- chore: add minutes for 2024-09-19 meeting ([#1035](https://github.com/nodejs/Release/pull/1035)) | ||
|
||
### readable-stream | ||
|
||
#### Open Issues | ||
|
||
- process is not defined ([#540](https://github.com/nodejs/readable-stream/issues/540)) | ||
|
||
### nodejs-dependency-vuln-assessments | ||
|
||
#### Open Issues | ||
|
||
- Plan to port the fixes to the openssl versions (CVE-2024-6119) in use for nodejs. ([#192](https://github.com/nodejs/nodejs-dependency-vuln-assessments/issues/192)) | ||
|
||
### node-auto-test | ||
|
||
#### Open Pull Requests | ||
|
||
- tools: add WPT updater for specific subsystems ([#73](https://github.com/nodejs/node-auto-test/pull/73)) | ||
|
||
### TSC | ||
|
||
#### Open Pull Requests | ||
|
||
- Choosing a marker to customize default exports returned by `require(esm)` ([#1622](https://github.com/nodejs/TSC/pull/1622)) | ||
- doc: add minutes for meeting Sep 18 2024 ([#1620](https://github.com/nodejs/TSC/pull/1620)) | ||
|
||
#### Merged Pull Requests | ||
|
||
- Start a vote on require(esm) markers ([#1621](https://github.com/nodejs/TSC/pull/1621)) | ||
|
||
### branch-diff | ||
|
||
#### Open Pull Requests | ||
|
||
- chore(deps): bump actions/setup-node from 4.0.3 to 4.0.4 ([#73](https://github.com/nodejs/branch-diff/pull/73)) | ||
|
||
### node-core-utils | ||
|
||
#### Open Pull Requests | ||
|
||
- fix: followup fix for supported wpt jsons ([#855](https://github.com/nodejs/node-core-utils/pull/855)) | ||
|
||
#### Merged Pull Requests | ||
|
||
- chore(wpt): warn for non-existent wpt path ([#856](https://github.com/nodejs/node-core-utils/pull/856)) |