-
Notifications
You must be signed in to change notification settings - Fork 27
Dependency on glob is unused, but creates warnings on install #547
Copy link
Copy link
Open
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Description
Environment details
- OS: all
- Node.js version: tested with v24
- npm version: tested with 11.11.0
code-suggesterversion: 5.0.1
Steps to reproduce
- Install
code-suggesteras a dependency in a project - Receive warning from npm:
npm warn deprecated glob@7.2.3: Glob versions prior to v9 are no longer supported
I found PR #359 in which any further updates to glob were silenced. The reason was that at that time, glob v8 (also deprecated by now) required at least Node 12.
However, in this project, there is no mention of glob outside the package*.json files.
Therefore, it should be removed.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.