gulpjs / gulpjs/vinyl-fs

Drop recursive readlink when we drop Node 10

Open
#337 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
973
Forks
154
PR merge metrics
No merged PRs in 30d

Description

Node 10 on Windows throws a EPERM when we `fs.stat` on a symlink to a directory. We work around this by recursively readlink on symlinks found by source, but we can remove this workaround when we drop Node 10 support.

_Originally posted by @phated in https://github.com/gulpjs/vinyl-fs/pull/333#discussion_r1225647122_

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the recursive readlink workaround used for symlinks found by source, then inspect any related tests. Confirm that Node 10 support has been dropped before removing the workaround, and run the relevant test suite to verify symlink handling still passes.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.