Add a few root files to lint-spell

[silverwind]

Files in root were not linted, add them. No new violations.

[silverwind]

Changed entry in ESLINT_FILES to *.js to match.

[silverwind]

Should we spellcheck CHANGELOG.md or do we consider it append-only? I'd say yes, we can change it.

[techknowlogick]

I'd say yes, we can change it.

I agree

[silverwind]

Applied to changelog as well now and ran make lint-spell-fix. Also the wildcard is removed, which will give better Makefile performance.

[silverwind]

Ideally there would be a way to also lint the PR title during PR checks, not sure how to do it though, at least not without a temp file and then there's the issue that CI does not run after commit title change.

[techknowlogick]

Ideally there would be a way to also lint the PR title during PR checks, not sure how to do it though, at least not without a temp file and then there's the issue that CI does not run after commit title change.

New @GiteaBot functionality? 🤔

[silverwind]

Ideally there would be a way to also lint the PR title during PR checks, not sure how to do it though, at least not without a temp file and then there's the issue that CI does not run after commit title change.

New @GiteaBot functionality? 🤔

Likely will be hard to run the go-based spellchecker in the bot which is typescript. If it can run the go checker, then it would likely be as simple as writ the string to a file, run on that and check the exit code.

cc: @yardenshoham.

[yardenshoham]

hmmmm, maybe with https://www.npmjs.com/package/cspell

[silverwind]

I think I may actually exclude changelog again because I don't want to put the burden onto the releaser to fix typos that they haven't made.

[silverwind]

hmmmm, maybe with https://www.npmjs.com/package/cspell

On topic: https://github.com/crate-ci/typos looks interesting, but it's rust which we have not integrated yet as tooling language.

[silverwind]

CHANGELOG.md is excluded again. I think we can at least keep the typo fixes in it, so it's a one-time correction basically. The wildcard does not seem to have any impact on Makefile performance, so I kept it.