Remove .pipe method
See original GitHub issueThis method seems like really old cruft that doesn’t belong.
A few reasons I think it should be removed:
- Vinyl isn’t actually a stream, so having a
pipe
method could confuse duck-typing - The pattern for dealing with streams in plugins is:
if (file.isStream()) {
file.contents = file.contents.pipe(transform());
}
@contra do you agree?
Issue Analytics
- State:
- Created 7 years ago
- Comments:5 (2 by maintainers)
Top Results From Across the Web
The Definitive Guide to Trenchless Pipeline Removal
Conventional pipeline removal techniques usually involve extensive excavation to expose the abandoned pipe. The pipe is then cut into ...
Read more >8.4. Pipe Removal Methods - Access Engineering
Pipe removal is a replacement technique, which breaks and removes the existing pipe fragments rather than displacing them into the surrounding ground as...
Read more >How to Remove Pipe Dope? (3 EASY METHODS)
Spray the soapy water on the joints if you want to find out the leaks. · Shut off the gas line and loosen...
Read more >3 Ways to REMOVE PVC Pipe From a FITTING (Video Review)
The first method is by using a pry bar. Roger demonstrates how to use the pry bar to gently lever the pipe out...
Read more >How To Remove An Old Threaded Pipe (Just 4 Steps!)
If there is no union handy, you'll need to take a more direct approach. Cut the pipe with a hacksaw fitted with a...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
@binarykitchen This ticket was for the removal of a really old function
file.pipe
that was never really used or adopted.file.contents = file.contents.pipe(whatever)
is the replacement for this case. There are a bunch of issues with your example code, but that’s more something you should get help with on StackOverflow or similar.Right, thanks