ESlint doesn't exit failure for warnings, there's a few warnings that made it through to master (JSDoc formatting), let's also add "--max-warnings 0" to our npm script to catch this going forward.
Alternatively, we can change all the rules to be errors instead of warnings.