DiffViewer - Add link to Bugzilla about firefox-code-coverage-frontend HOT 9 CLOSED

Here is an example link https://bugzilla.mozilla.org/show_bug.cgi?id=1429461

from firefox-code-coverage-frontend.

This json-diff view might help: https://hg.mozilla.org/mozilla-central/json-diff/8cde429207f3/layout/reftests/css-ui-valid/select/reftest.list

from firefox-code-coverage-frontend.

I would like to get more info from Calixte wrt to the actual pre-filled comment format and how does that work.

from firefox-code-coverage-frontend.

I think it isn't possible to add a pre-filled comment (in fact it's only possible to have a pre-fiiled comment #0 when you create a bug).
So maybe just having something to copy-paste in the bug.

from firefox-code-coverage-frontend.

@calixteman what would you like the text to be copied to look like?

@klahnakoski This issue could take most of the UCOSP term for one student.

I believe that we can add a comment via the BZ rest API:
https://wiki.mozilla.org/Bugzilla:REST_API
http://bugzilla.readthedocs.io/en/latest/api/core/v1/comment.html#create-comments

I believe @canuckistani would still recommend using his wrapper:
https://github.com/canuckistani/bz.js
https://www.npmjs.com/package/bz
Worth contacting him.

Making comments on BZ can be tested against https://landfill.bugzilla.org/bugzilla-5.0-branch/
It is probably best to talk with dylan and/or blob.
Maybe the place to test is against https://api-dev.bugzilla.mozilla.org

Also notice if the user has 2-step authentication:

If you have two factor authentication on your Bugzilla account (which is recommended),
then you will not be able to call the login endpoint. Instead create a new API key. Then
that API key can be passed to each request.

from firefox-code-coverage-frontend.

This is probably lower priority than other items.

from firefox-code-coverage-frontend.

Maybe a good way could be to add coverage info in a comment in using Marco's addon:
https://github.com/marco-c/code-coverage-addon

from firefox-code-coverage-frontend.

Should we then close this issue?
We can revisit in the future if the addon is not sufficient.

from firefox-code-coverage-frontend.

Sounds good.

from firefox-code-coverage-frontend.