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 isssues #26

Open
jsachs opened this issue Feb 5, 2016 · 1 comment
Open

jshint isssues #26

jsachs opened this issue Feb 5, 2016 · 1 comment

Comments

@jsachs
Copy link
Contributor

jsachs commented Feb 5, 2016

I get the following when I run jshint on the ntrlab branch. Are these the result of incomplete code?

app/controllers/surveys.js
  line 258  col 31  'question' used out of scope.
  line 259  col 21  'question' used out of scope.
  line 264  col 66  'question' used out of scope.
  line 268  col 27  'q' is already defined.
  line 272  col 66  'question' used out of scope.
  line 277  col 34  'q' used out of scope.

app/controllers/tasks.js
  line 126  col 1   A generator function shall contain a yield statement.
jsachs added a commit that referenced this issue Feb 5, 2016
- linting and beautification
- there are a few outstanding linting errors, noted in issue #26
jsachs added a commit that referenced this issue Feb 5, 2016
- linting and beautification
- there are a few outstanding linting errors, noted in issue #26
@jsachs
Copy link
Contributor Author

jsachs commented Feb 5, 2016

I also recommend that you rebase from develop on a daily basis, to avoid repeating linting/merging issues.

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