[jira] [Commented] (MESOS-7924) Add a javascript linter to the webui.

["Armand Grillet (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MESOS-7924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16155339#comment-16155339 ] 

Armand Grillet commented on MESOS-7924:
---------------------------------------

ESLint should be installed [using npm|https://eslint.org/docs/user-guide/getting-started] which is currently not used to install other dependencies. The two linters we currently use are cpplint ([thanks to this script|https://github.com/apache/mesos/blob/7b5b728da41e0ebecca50a43cba80e680bebb37e/support/cpplint.py]) and pylint (installed using pip). 

Setting up ESLint should be straightforward but I am not sure what is the best way to add this new linter as a dependency, any recommendations?

> Add a javascript linter to the webui.
> -------------------------------------
>
>                 Key: MESOS-7924
>                 URL: https://issues.apache.org/jira/browse/MESOS-7924
>             Project: Mesos
>          Issue Type: Improvement
>          Components: webui
>            Reporter: Benjamin Mahler
>              Labels: tech-debt
>
> As far as I can tell, javascript linters (e.g. ESLint) help catch some functional errors as well, for example, we've made some "strict" mistakes a few times that ESLint can catch: MESOS-6624, MESOS-7912.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)