yahoo-eng-team team mailing list archive

[Itxaka Serrano <iserrano@xxxxxxxxxx>]

Public bug reported:

The eslint package is frozen to version 1.2.1, which was released the
20th of August of 2015.

The newer versions (last 1.x version is 1.10.3) fixes a lot of issues
and a lot of checks which are not working correctly or which are totally
missing:

http://eslint.org/docs/rules/operator-linebreak.html

Which is activated on eslint-config-openstack but was never checked:
https://github.com/openstack/horizon/blob/master/horizon%2Fstatic%2Fhorizon%2Fjs%2Fhorizon.quota.js#L103


Also it provides a new config flag "--max-warnings" which would be incredibly useful so we can put a limit to the number of warning that we get and enforce a new check.

** Affects: horizon
     Importance: Undecided
     Assignee: Itxaka Serrano (itxakaserrano)
         Status: New

** Changed in: horizon
     Assignee: (unassigned) => Itxaka Serrano (itxakaserrano)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1547406

Title:
  eslint should be upgraded

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  The eslint package is frozen to version 1.2.1, which was released the
  20th of August of 2015.

  The newer versions (last 1.x version is 1.10.3) fixes a lot of issues
  and a lot of checks which are not working correctly or which are
  totally missing:

  http://eslint.org/docs/rules/operator-linebreak.html

  Which is activated on eslint-config-openstack but was never checked:
  https://github.com/openstack/horizon/blob/master/horizon%2Fstatic%2Fhorizon%2Fjs%2Fhorizon.quota.js#L103

  
  Also it provides a new config flag "--max-warnings" which would be incredibly useful so we can put a limit to the number of warning that we get and enforce a new check.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1547406/+subscriptions