3
100%
master: 100%

Ran 05 Feb 2019 06:11PM UTC

Files 15

Run time 1min

Badge

Embed ▾

Committed 05 Feb 2019 06:07PM UTC coverage: 98.471% (+0.02%) from 98.448%

Job # 2908.3

Build Type

push

travis-ci

Committed by

rwaldron

Commit Message

[[FIX]] Do not add cls method names to env. record

A recent refactoring of the class parsing logic [1] removed the function
name inference logic (i.e. integration with the `nameStack` object) in
favor of a similar codepath within the generic `doFunction` routine. By
providing the method name to that routine, the change had an unintended
side effect: method names were used to create new entries in the new
environment record. This was observable from user code in the presence
of JSHint's `unused` linting option: when set, methods would incorrectly
shadow variables that shared the same name.

Continue to rely on the `doFunction` method for name inference (limiting
duplication), but disable the extension of the environment record for
that case.

Run Details

4379 of 4447 relevant lines covered (98.47%)

3813.14 hits per line

jshint / jshint / 2908 / 3
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2908.3

jshint / jshint / 2908 / 3 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2908.3

jshint / jshint / 2908 / 3
100%
master: 100%

README BADGES
x