Merge pull request #18 from mutablelogic/dev2 #9
djthorpeAnnotations
10 errors and 2 warnings
|
docs:
src/canvas/ContentElement.js#L2
'nothing' is defined but never used
|
|
docs:
src/canvas/ContentElement.js#L30
Expected 'this' to be used by class method 'render'
|
|
docs:
src/core/ProviderElement.js#L80
Assignment to function parameter 'path'
|
|
docs:
src/core/ProviderElement.js#L83
Assignment to function parameter 'request'
|
|
docs:
src/core/ProviderElement.js#L86
Assignment to function parameter 'interval'
|
|
docs:
src/element/TableHeadElement.js#L59
The body of a for-in should be wrapped in an if statement to filter unwanted properties from the prototype
|
|
docs:
src/element/TableHeadElement.js#L59
for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array
|
|
docs:
src/geojson/FeatureModel.js#L1
Unable to resolve path to module '../core/Model'
|
|
docs:
src/geojson/FeatureModel.js#L1
Missing file extension for "../core/Model"
|
|
docs:
src/geojson/ShapeModel.js#L1
Unable to resolve path to module '../core/Model'
|
|
docs:
src/mapbox/MapElement.js#L145
Unexpected console statement
|
|
docs:
src/mapbox/MapElement.js#L146
Unexpected console statement
|