Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jshint warnings #44

Open
dduugg opened this issue Feb 20, 2015 · 0 comments
Open

jshint warnings #44

dduugg opened this issue Feb 20, 2015 · 0 comments

Comments

@dduugg
Copy link

dduugg commented Feb 20, 2015

jshintrc should be named .jshintrc. With that change, jshint will generate the following warnings:

$ jshint src/js/pebble-js-app.js
src/js/pebble-js-app.js: line 200, col 18, 'MessageQueue' was used before it was defined.
src/js/pebble-js-app.js: line 11, col 23, 'window' is not defined.
src/js/pebble-js-app.js: line 51, col 17, 'clearTimeout' is not defined.
src/js/pebble-js-app.js: line 273, col 25, 'toString' is not defined.
src/js/pebble-js-app.js: line 296, col 21, 'clearTimeout' is not defined.
src/js/pebble-js-app.js: line 307, col 21, 'clearTimeout' is not defined.
src/js/pebble-js-app.js: line 358, col 25, 'window' is not defined.

7 errors

Happy to submit a PR to resolve the above. I can also include a Travis configuration to catch future warnings.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/8752532-jshint-warnings?utm_campaign=plugin&utm_content=tracker%2F2453141&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2453141&utm_medium=issues&utm_source=github).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant