import-js/eslint-plugin-import
`no-restricted-paths` should support negated glob pattern
开放
#2,800 创建于 2023年6月16日
help wanted
仓库指标
- 星标
- (5,938 个星标)
- PR 合并指标
- (平均合并 138天 22小时) (30 天内合并 3 个 PR)
描述
Bug Description
We cannot use a negated glob pattern (e.g. '!dir_to_exclude/**') in zone.target, zone.from, and zone.except.
Impact
This prevents us from specifying a target that matches anything except certain files or directories. Fixing the bug should help address the use case requested in #2497 as well.
Root Cause
This is due to path.resolve is used to resolve the glob patterns like a relative path (e.g. '!dir_to_exclude/**' becomes '/home/project_dir/!dir_to_exclude/**'), making it invalid.
Suggested Fix
I'd love to suggest the following fix:
- When constructing a minimatch matcher, instead of resolving the glob patterns as paths, the glob patterns shall be passed as is to minimatch, and
- When matching a path, instead of using the absolute path, the path should be resolved as a relative path to the base path specified in
base.