New files are only detect after deleting `.eslint-meteor-files`

quavedev / eslint-plugin

4 stars 2 forks source link

New files are only detect after deleting `.eslint-meteor-files` #8

Open filipenevola opened 3 months ago

filipenevola commented 3 months ago

I was working on a project and noticed some rules were not working.

They should have returned the error to me, but they did not.

Then I realized by debugging (METEOR_ESLINT_PLUGIN_DEBUG=1 eslint .) that new files were not applying the new rules as they are not present on .eslint-meteor-files.

filipenevola commented 3 months ago

@renanccastro, what is the best way to handle this issue?

My initial ideas

Delete the cache .eslint-meteor-files when it's older than 5 seconds or so
Detect new file and then delete .eslint-meteor-files

filipenevola commented 3 months ago

I implemented the expires logic for now https://github.com/quavedev/eslint-plugin/blob/main/CHANGELOG.md#142---2024-03-22

renanccastro commented 3 months ago

The best way would be to add a watcher on folders, and re-run the tree logic when something is added.