tools: `make lint-js` should not lint test/tmp*

[bnoordhuis]

test/tmp (and test/tmp.0, test/tmp.1, etc.) sometimes have temporary files in them with a .js or .mjs suffix. The linter should not lint those. =)

cc @joyeecheung

[priyank-p]

@bnoordhuis will adding test/temp and test/temp.* fix it?

[vorontsov-pavel]

@cPhost yes, I tried. PS: excluding test/.tmp*

[Trott]

If you have a test/tmp, it is leftover from a long-ish time ago. (EDIT: Well, a few weeks ago at least.) We changed it to .tmp (leading .) to work around a issue with Windows/globbing/.eslintignore.

So just remove your test/tmp* directories and they should never be re-created and everything will be awesome.

[Trott]

I'm going to close this, but feel free to re-open if you think that's wrong-headed.

[Trott]

Here's the PR that introduced the change. It landed 19 days ago so not a
long time ago but not terribly recent either. #16372

[joyeecheung]

I have not checked if the tmp directories are ignored by git, but git clean -xfd ./test is probably a best practice when switching between branches..