Since bug 1151765 has landed and support es6 syntax for jsdoc parsing, there're still some syntax issues not resolved in gaia. We could solve jsdoc issues per-app.
Assigning to Nils for the MozWeekend in Berlin
Assignee: nobody → nils
Whiteboard: good-first-bug, zh → good-first-bug, zh, mozweekend
I could not reproduce any syntax errors when running `gulp jsdoc:keyboard`. The only issues were warnings about the ES6 arrow expressions, like this: "WARNING: JSDoc does not currently handle ArrowFunctionExpression nodes." Also, the generated documentation does look okay. What errors is this ticket about and how can they be produced?
Status: NEW → ASSIGNED
Thats normal. Please check apps/keyboard/jsdoc.json file and remove the files in exclude field.
@nils I think, what gasolin is trying to say is, the problematic files are excludes from JSDoc generation by now. If you remove them from the exclude field in the mentioned jsdoc.json, you should see the the syntax errors. Btw, was nice to meet you again! :)
Fred, after remove the files in exclude field, no errors were found. I tested it and created a patch. Setting r? to a Keyboard owner module to see if this could be merged.
Comment on attachment 8638122 [details] [review] [gaia] Gioyik:bug-1170460 > mozilla-b2g:master I am not familiar w/ jsdocs related work.
Attachment #8638122 - Flags: review?(timdream) → review?(gasolin)
Comment on attachment 8638122 [details] [review] [gaia] Gioyik:bug-1170460 > mozilla-b2g:master Tests fine. Thanks for contribution!
Attachment #8638122 - Flags: review?(gasolin) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.