Commit 650c9bd
stream: pipeline with end option
Currently pipeline cannot fully replace pipe due
to the missing end option. This PR adds the end
option to the promisified pipeline method.
PR-URL: #40886
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 534409d commit 650c9bdCopy full SHA for 650c9bd
File tree
Expand file treeCollapse file tree
3 files changed
+50
-17
lines changedOpen diff view settings
Filter options
- lib
- internal/streams
- stream
- test/parallel
Expand file treeCollapse file tree
3 files changed
+50
-17
lines changedOpen diff view settings
Collapse file
lib/internal/streams/pipeline.js
Copy file name to clipboardExpand all lines: lib/internal/streams/pipeline.js+25-15Lines changed: 25 additions & 15 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
109 | 109 | |
110 | 110 | |
111 | 111 | |
112 | | - |
| 112 | + |
113 | 113 | |
114 | 114 | |
115 | 115 | |
| ||
153 | 153 | |
154 | 154 | |
155 | 155 | |
156 | | - |
| 156 | + |
| 157 | + |
| 158 | + |
157 | 159 | |
158 | 160 | |
159 | 161 | |
| ||
227 | 229 | |
228 | 230 | |
229 | 231 | |
| 232 | + |
230 | 233 | |
231 | 234 | |
232 | | - |
233 | | - |
234 | | - |
235 | | - |
236 | | - |
237 | | - |
238 | | - |
239 | | - |
240 | | - |
| 235 | + |
| 236 | + |
| 237 | + |
| 238 | + |
| 239 | + |
| 240 | + |
| 241 | + |
| 242 | + |
| 243 | + |
| 244 | + |
| 245 | + |
| 246 | + |
| 247 | + |
241 | 248 | |
242 | 249 | |
243 | 250 | |
| ||
282 | 289 | |
283 | 290 | |
284 | 291 | |
285 | | - |
| 292 | + |
| 293 | + |
| 294 | + |
| 295 | + |
286 | 296 | |
287 | 297 | |
288 | 298 | |
289 | 299 | |
290 | 300 | |
291 | 301 | |
292 | | - |
| 302 | + |
293 | 303 | |
294 | 304 | |
295 | 305 | |
| ||
302 | 312 | |
303 | 313 | |
304 | 314 | |
305 | | - |
| 315 | + |
306 | 316 | |
307 | 317 | |
308 | 318 | |
| ||
314 | 324 | |
315 | 325 | |
316 | 326 | |
317 | | - |
| 327 | + |
318 | 328 | |
319 | 329 | |
320 | 330 | |
|
Collapse file
+3-1Lines changed: 3 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
16 | 16 | |
17 | 17 | |
18 | 18 | |
| 19 | + |
19 | 20 | |
20 | 21 | |
21 | 22 | |
22 | 23 | |
23 | 24 | |
| 25 | + |
24 | 26 | |
25 | 27 | |
26 | 28 | |
| ||
29 | 31 | |
30 | 32 | |
31 | 33 | |
32 | | - |
| 34 | + |
33 | 35 | |
34 | 36 | |
35 | 37 | |
|
Collapse file
test/parallel/test-stream-pipeline.js
Copy file name to clipboardExpand all lines: test/parallel/test-stream-pipeline.js+22-1Lines changed: 22 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
1465 | 1465 | |
1466 | 1466 | |
1467 | 1467 | |
1468 | | - |
| 1468 | + |
| 1469 | + |
| 1470 | + |
| 1471 | + |
| 1472 | + |
| 1473 | + |
| 1474 | + |
| 1475 | + |
| 1476 | + |
| 1477 | + |
| 1478 | + |
| 1479 | + |
| 1480 | + |
| 1481 | + |
| 1482 | + |
| 1483 | + |
| 1484 | + |
| 1485 | + |
| 1486 | + |
| 1487 | + |
| 1488 | + |
| 1489 | + |
1469 | 1490 | |
0 commit comments