Hound warnings on JS test files
Created by: maxbeatty
I've opened a few PRs and each gets lots of comments from @houndci-bot because the JS unit test files do not conform to the ESLint settings. Here are few ways I've considered addressing this:
- Ignore the test files similar to #20446. This is the least amount of work and will get other PRs passing and more likely to merge.
- Fix the eslint issues in the test files, add a grunt task or npm script (#20332) to transpile the test files, and update the references in
index.html
. This is obviously more work but would be more of a long term benefit.
I'm open to other ways to please @houndbot-ci when changing JS test files and willing to put in the work