Skip to content

Fix file watching#1376

Open
segevfiner wants to merge 2 commits intoegoist:mainfrom
segevfiner:fix-file-watching
Open

Fix file watching#1376
segevfiner wants to merge 2 commits intoegoist:mainfrom
segevfiner:fix-file-watching

Conversation

@segevfiner
Copy link
Copy Markdown
Contributor

The performance of using tinyglobby to figure out ignores was just way too slow as it gets queried for every single file under the working directory including node_modules, replace with a glob-to-regex.js which is just a regex check with no file system access and/or .gitignore parsing which is not needed here

Hopefully fixes #1293 and others

The performance of using tinyglobby to figure out ignores was just way too slow as it gets queried for every single file under the working directory including `node_modules`, replace with a glob-to-regex.js which is just a regex check with no file system access and/or .gitignore parsing which is not needed here
@codesandbox
Copy link
Copy Markdown

codesandbox bot commented Nov 18, 2025

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@vercel
Copy link
Copy Markdown

vercel bot commented Nov 18, 2025

@segevfiner is attempting to deploy a commit to the EGOIST's projects Team on Vercel.

A member of the Team first needs to authorize it.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Nov 18, 2025

Open in StackBlitz

npm i https://pkg.pr.new/egoist/tsup@1376

commit: da4eea8

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedglob-to-regex.js@​1.2.010010010085100

View full report

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.

Watch doesn't work anymore since upgrade to 8.3.5 / 8.3.6 (working in 8.3.0)

1 participant