#3674 - Add a unit test to run ESLint
| Identifier | #3674 |
|---|---|
| Issue type | Feature request or suggestion |
| Title | Add a unit test to run ESLint |
| Status | Completed |
| Tags |
Roadmap: Over the horizon (custom) |
| Handling member | Chris Graham |
| Addon | General / Uncategorised |
| Description | ESLint configuration files were recently added to the repo[1]. We should have a unit test that runs ESLint and outputs any errors.
1: https://github.com/ocproducts/composr/commit/a14e994ba3d1eb5142d33f710214dfc3fc03c22c |
| Steps to reproduce | |
| Funded? | No |
The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".

Comments
1) It should be '_' prefixed, i.e. a test you're not supposed to run for every patch release and doesn't need to be 100% respected
2) It should not just crash if eslint and whatever dependencies it has are not available. I imagine that needs node or some such, which I don't want core developers to be forced to install. So it needs to do a clean assertion error when eslint is unavailable.
Sorry for making this suggestion without thinking about it fully...
However, I don't think it needs to be for now at least.