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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(core): handle paths deleted by the ide #26363

Merged
merged 1 commit into from
Jun 5, 2024
Merged

fix(core): handle paths deleted by the ide #26363

merged 1 commit into from
Jun 5, 2024

Conversation

FrozenPandaz
Copy link
Collaborator

@FrozenPandaz FrozenPandaz commented Jun 4, 2024

Current Behavior

When IDEs delete directories on some operating systems, the watch events are registered as deleted paths. These events were not handled so plugins were continuing to process files which did not exist.

Expected Behavior

Watch events which are directories being deleted are handled and plugins will not continue to process files which do not exist.

Related Issue(s)

Fixes #

@FrozenPandaz FrozenPandaz requested review from a team as code owners June 4, 2024 18:58
@FrozenPandaz FrozenPandaz requested a review from AgentEnder June 4, 2024 18:58
Copy link

vercel bot commented Jun 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Jun 4, 2024 8:14pm

@FrozenPandaz FrozenPandaz self-assigned this Jun 4, 2024
@FrozenPandaz FrozenPandaz merged commit 7e984e1 into master Jun 5, 2024
6 checks passed
@FrozenPandaz FrozenPandaz deleted the fix-dir branch June 5, 2024 20:38
Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants