Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

consider switching to brace-expansion #1301

Open
43081j opened this issue Feb 4, 2024 · 4 comments
Open

consider switching to brace-expansion #1301

43081j opened this issue Feb 4, 2024 · 4 comments

Comments

@43081j
Copy link

43081j commented Feb 4, 2024

馃憢 as part of the efforts going on over at the ecosystem-cleanup repo, we're helping projects move away from various packages to reduce dependency bloat

braces is one such package.

in most places, we can use brace-expansion as a drop-in replacement instead (4-5x smaller, only 1 dependency)

i'd be happy to open a PR with the change if you're happy with it

@paulmillr
Copy link
Owner

It's probably much slower.

@dave-addition
Copy link

Sort of related: CVE-2024-4068 on braces was just made public (see also: micromatch/braces#35).

Hopefully that project fixes it, but... last publish was 5 years ago so we'll see.

@paulmillr
Copy link
Owner

tired of these useless "vulnerabilities"

@dave-addition
Copy link

I sympathize!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants