node icon indicating copy to clipboard operation
node copied to clipboard

src: disable fast methods for `buffer.write`

Open targos opened this issue 1 year ago • 13 comments

It should resolve the regressions while we work on fixing them.

Refs: https://github.com/nodejs/node/issues/54521

targos avatar Aug 26 '24 06:08 targos

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.33%. Comparing base (1399d4e) to head (a7a10a7). Report is 323 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #54565      +/-   ##
==========================================
- Coverage   87.33%   87.33%   -0.01%     
==========================================
  Files         649      649              
  Lines      182626   182620       -6     
  Branches    35041    35040       -1     
==========================================
- Hits       159503   159489      -14     
- Misses      16394    16404      +10     
+ Partials     6729     6727       -2     
Files with missing lines Coverage Δ
src/node_buffer.cc 69.94% <100.00%> (-0.71%) :arrow_down:

... and 23 files with indirect coverage changes

codecov[bot] avatar Aug 26 '24 07:08 codecov[bot]

If this is accepted, it should be fast-tracked for inclusion in https://github.com/nodejs/node/pull/54560

targos avatar Aug 26 '24 07:08 targos

Fast-track has been requested by @targos. Please 👍 to approve.

github-actions[bot] avatar Aug 26 '24 07:08 github-actions[bot]

@bricss please do not approve fast-track requests :] (or you know, start making PRs and join as a collaborator that also works :))

benjamingr avatar Aug 26 '24 10:08 benjamingr

CI: https://ci.nodejs.org/job/node-test-pull-request/61482/

nodejs-github-bot avatar Aug 26 '24 12:08 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/61487/

nodejs-github-bot avatar Aug 26 '24 13:08 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/61509/

nodejs-github-bot avatar Aug 26 '24 18:08 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/61524/

nodejs-github-bot avatar Aug 27 '24 03:08 nodejs-github-bot

@targos can you fix commit msg (PR-URL)?

ronag avatar Aug 27 '24 04:08 ronag

@ronag I don't see a problem in the commit message.

targos avatar Aug 27 '24 15:08 targos

CI: https://ci.nodejs.org/job/node-test-pull-request/61563/

nodejs-github-bot avatar Aug 27 '24 16:08 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/61569/

nodejs-github-bot avatar Aug 27 '24 19:08 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/61571/

nodejs-github-bot avatar Aug 27 '24 21:08 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/61586/

nodejs-github-bot avatar Aug 28 '24 13:08 nodejs-github-bot

@ronag I don't see a problem in the commit message.

@targos

Commit message must match a given regex pattern: PR-URL: https://github.com/(nodejs|nodejs-private)/(node|node-private)/pull/\d+

ronag avatar Aug 28 '24 13:08 ronag

This message appears (to collaborators) on all the pull requests of the project. Unfortunately there is no way to customize it but the goal is to prevent merging by accident using the GitHub UI.

targos avatar Aug 28 '24 13:08 targos

CI: https://ci.nodejs.org/job/node-test-pull-request/61593/

nodejs-github-bot avatar Aug 28 '24 15:08 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/61601/

nodejs-github-bot avatar Aug 28 '24 19:08 nodejs-github-bot

CI: https://ci.nodejs.org/job/node-test-pull-request/61616/

nodejs-github-bot avatar Aug 28 '24 23:08 nodejs-github-bot

Landed in 7616855b2a90e03310ef8431274ffa37d78abe68

nodejs-github-bot avatar Aug 29 '24 07:08 nodejs-github-bot