[SERVER-25617] Make scons lint include non-jstests js files Created: 15/Aug/16 Updated: 15/Aug/16 Resolved: 15/Aug/16 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Testing Infrastructure |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Scott Hernandez (Inactive) | Assignee: | DO NOT USE - Backlog - Test Infrastructure Group (TIG) |
| Resolution: | Incomplete | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Participants: |
| Description |
|
There are files in mongo/shell (like replsettest.js) which aren't checked when eslint is run, and they should be. We may also want to make this a requirement when compiling the shell/mongo to catch any js format/syntax problems which would only surface at runtime. |
| Comments |
| Comment by Scott Hernandez (Inactive) [ 15/Aug/16 ] |
|
Turns out it does this already – not the part where shell target requires it, but lint does it. |