Conversation
- `CopyOptions` on `cp` now defaults to empty - Setting option values is easier now
packages/io/src/io-util.ts
Outdated
|
|
||
| // remove redundant slashes | ||
| const isUnc = /^\\\\+[^\\]/.test(p) // e.g. \\hello | ||
| return (isUnc ? '\\' : '') + p.replace(/\\\\+/g, '\\') // preserve leading // for UNC |
There was a problem hiding this comment.
UNC paths are paths to shared folders - its a Windows concept.
There was a problem hiding this comment.
UNC shares are mostly a concept for on-prem server (TFS). Not sure what it means is and when it's GHE.
For cloud, there is private agents and azure express routes. Both are not relevant right now for this effort but might be in the future. I would be OK with stripping the concepts for now and see if and when they come back.
There was a problem hiding this comment.
Makes sense, went ahead and cut it
Make packages/io async
Create an optimistic mkdirp
|
I'm 👍 to merge this—there are some more improvements we can make (specifically, for example, stuff like the building of the executable extension lift can probably go in @damccorm Let me know if you're good to go, or if there are more changes you'd like to make, first. |
3 participants
No description provided.