1170456 - [Calendar] fix remain jsdoc error

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Calendar, 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

[Romain Loisel]

I would like to work on it, can you assigned me on it ?

Comment 2

[[e-mail]]

Thanks for interesting in solving this issue.

First you need run `npm install` in gaia repo to get up-to-date packages. 

Then run command

`gulp jsdoc:calendar` to make sure everything works.

You can remove exclude files from jsdoc.json, fix the file and run the command again to check if jsdoc is actually fixed.

Comment 3

[Romain Loisel]

Attachment: List of the warning from JSDoc output on calendar

OK, i succeed to launch JSDoc on calendar, all of the warnings comes from non-support ES6 in JSDoc.

Do you have an idea how i can manage these warnings ?

Comment 4

[[e-mail]]

Warning is not relevant. 

Please try to remove exclude files from calendar's jsdoc.json, and see if any error showed in console log.

Comment 5

[Romain Loisel]

Attachment: PR, review

Bug 1170456 - [Calendar] fix remain jsdoc error #30710
"https://github.com/mozilla-b2g/gaia/pull/30710"

Comment 6

[[e-mail]]

Comment on attachment 8626207 [details] [review]
PR, review

Thanks for provide fixes, the jsdoc fix part looks good to me.

Regarding the maintenance part, Please use rebase instead of merge so there's no extra merge log in gaia tree. And please squash your PR to single commit.

After fixing the issue, you could request review with [e-mail] who is the module owner of calendar app.
https://wiki.mozilla.org/Modules/FirefoxOS

Comment 7

[Romain Loisel]

Attachment: Bug 1170456 - [Calendar] fix remain jsdoc error R=romainloisel #30724

Bug 1170456 - [Calendar] fix remain jsdoc error R=romainloisel #30724 
"https://github.com/mozilla-b2g/gaia/pull/30724"

Comment 9

[Romain Loisel]

Gaye, could you review my PR ?

Comment 10

[[e-mail]]

Comment on attachment 8626599 [details] [review]
Bug 1170456 - [Calendar] fix remain jsdoc error R=romainloisel #30724

gaye is now out of office, transfer the review to Miller.

Comment 11

[Miller Medeiros [:millermedeiros]]

Comment on attachment 8626599 [details] [review]
Bug 1170456 - [Calendar] fix remain jsdoc error R=romainloisel #30724

landed on master, sorry for the huge delay on the review: https://github.com/mozilla-b2g/gaia/commit/6dfeea505ce6efb3d139402c33b0695e247df9b6