You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is a very significant major version upgrade from v7 to v12, which includes a complete rewrite of the library and introduces numerous breaking changes. Code modifications are required for this upgrade.
Key Breaking Changes:
Promise-Based API: The most critical change is the move from a callback-based API to a modern Promise-based API starting in v9.0. Asynchronous operations now return Promises, and the old callback pattern is no longer supported.
Node.js Requirement: Support for older Node.js versions has been dropped. Version 9 and higher require Node.js v16 or newer.
Path Separators (Windows): Backslashes (\) are no longer treated as path separators in glob patterns; they are now exclusively used for escaping characters. All patterns, including on Windows, must use forward-slashes (/).
Module Exports: The package no longer has a default export. You must use named imports, for example: const { glob } = require('glob');.
Removed Options: Many options from v7 have been removed in v8.0, including silent, strict, nonull, nounique, and nosort. The behavior associated with these options (like ensuring unique results) is now standard.
Recommendation:
Due to the complete API rewrite, this upgrade requires careful code refactoring. You must update all glob calls to use the new Promise-based syntax and ensure your patterns use forward-slashes. Thorough testing is essential.
Notice 🤖: This content was augmented using artificial intelligence. AI-generated content may contain errors and should be reviewed for accuracy before use.
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You have signed the CLA already but the status is still pending? Let us recheck it.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):
package.jsonVulnerabilities that will be fixed with an upgrade:
SNYK-JS-BRACEEXPANSION-17706650
Breaking Change Risk
Important
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.