Skip to content

chore: switch to tinyglobby#574

Closed
benmccann wants to merge 3 commits into
floydspace:masterfrom
benmccann:tinyglobby
Closed

chore: switch to tinyglobby#574
benmccann wants to merge 3 commits into
floydspace:masterfrom
benmccann:tinyglobby

Conversation

@benmccann
Copy link
Copy Markdown

@floydspace
Copy link
Copy Markdown
Owner

hi @benmccann
thanks for contribution, could you pls explain what motivated you to make this fix
and pls apply prettier formater changes properly

@benmccann
Copy link
Copy Markdown
Author

My motivation was to reduce the number of dependencies. This reduces install size, but more importantly reduces supply chain risk. There are 14 people who have access to publish globby and its dependencies vs only 6 to publish tinyglobby and its dependencies because it has so many fewer dependencies. serverless-esbuild has 137 dependencies currently, but it could be trimmed down substantially with only a few simple changes such as this one and upgrading from chokidar 3 to chokidar 4.

@floydspace
Copy link
Copy Markdown
Owner

floydspace commented Jul 5, 2025

I don't think this switch will solve the issues you are describing, because globby is already a dependency of the serverless@3 package itself, which anyway has to be installed by end user.
so, in fact, total, with this change end user will have + one more + 2 dependencies.

@benmccann
Copy link
Copy Markdown
Author

Makes sense. I guess it would only make sense when supporting serverless@4

@benmccann benmccann closed this Jul 10, 2025
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

Successfully merging this pull request may close these issues.

2 participants