1170460 - [Keyboard] fix remain jsdoc error

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Keyboard, defect)

Description

[[e-mail]]

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.

Comment 1

[Michael Kohler [:mkohler]]

Assigning to Nils for the MozWeekend in Berlin

Comment 2

[nils]

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?

Comment 3

[[e-mail]]

Thats normal. Please check apps/keyboard/jsdoc.json file and remove the files in exclude field.

Comment 4

[André Fiedler]

@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! :)

Comment 5

[GitHub Autolander Bot]

Attachment: [gaia] Gioyik:bug-1170460 > mozilla-b2g:master

Comment 6

[Giovanny Gongora [:gioyik]]

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 7

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 8638122 [details] [review]
[gaia] Gioyik:bug-1170460 > mozilla-b2g:master

I am not familiar w/ jsdocs related work.

Comment 8

[[e-mail]]

Comment on attachment 8638122 [details] [review]
[gaia] Gioyik:bug-1170460 > mozilla-b2g:master

Tests fine. Thanks for contribution!

Comment 9

[[e-mail]]

merged https://github.com/mozilla-b2g/gaia/commit/a0e9f965b62a72df2a926d40b192db2960d9466c

Thanks!