Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Deprecate in favor of fb-watchman or chokidar #135

Open
aminya opened this issue Mar 31, 2021 · 0 comments
Open

Deprecate in favor of fb-watchman or chokidar #135

aminya opened this issue Mar 31, 2021 · 0 comments

Comments

@aminya
Copy link
Contributor

aminya commented Mar 31, 2021

Summary

Atom's Pathwatcher is non-recursive, has issues (e.g slow Directory.getEntries), and lacks a maintainer to address these. We should probably deprecate this library and switch to one of the alternatives:

  • fb-watchman: a native library that supports recursive watching. Being native means it is probably faster than Chokidar.
    fb-watchman

  • Chokidar is used in Microsoft's Visual Studio Code, gulp, karma, PM2, browserify, webpack, BrowserSync, and many others.
    chokidar

Motivation

Fixing issues with directories and path watching

Describe alternatives you've considered

Improving Pathwatcher. However, due to the fact that no one actually maintains this repository, I think it is pointless to spend time improving the repository.

Two of my PRs and many of the other PRs are left untouched.
https://github.com/atom/node-pathwatcher/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-desc

Additional context

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

No branches or pull requests

1 participant