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: respect the same ignore setting passed to chokidar #227

Closed

Conversation

DesignByOnyx
Copy link

This fixes #221.

@DesignByOnyx
Copy link
Author

Regarding the failing test - I am not able to reproduce on my machine (Ubuntu 20). pnpm test passes without issue. I hate to be the "works on my machine" guy, but I'm not sure how to proceed.

The micromatch library supports node >=8.6 and is written in CJS, so I don't think any of my changes really affect the one failing test:

image

@privatenumber
Copy link
Owner

No worries, it's a flaky test.

Will look at this more closely later, thanks for the PR.

@privatenumber
Copy link
Owner

Switching to anymatch as that's what chokidar uses.

@privatenumber
Copy link
Owner

I'm not sure if this actually fixes the problem you're describing in the issue.

I'm skeptical if chokidar will watch a file that is configured to be ignored even if it's passed in.

@DesignByOnyx
Copy link
Author

Please see my reply on the linked issue: #221 (comment)

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.

TSX is not ignoring symlinked dependencies in monorepo
2 participants