gulpjs / gulpjs/vinyl-fs

Remove normalization of file paths into globs

Open
#346 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

Everything below vinyl-fs expects globs to use `/` instead of `\\` (since `\\` is for escaping). However, I believe both the latest major versions of `vinyl-fs` and `glob-watcher` accept `\\` in globs.

We should ensure both projects only accept `/` for globs. This also requires separating globs from file paths in our test constants.

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the code that normalizes file paths into globs and the test constants mentioned in the issue. Separate glob constants from file-path constants, then run the existing test suite to verify that globs use `/` and that the intended path behavior is preserved.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.