Documentation fix patches

patrick at cloke.us patrick at cloke.us
Fri Jun 10 21:43:02 UTC 2016


That seems like a reasonable rule to me! We should strive to land code with documentation in the first place.

--Patrick



From: R Kent James
Sent: Friday, June 10, 2016 5:12 PM
To: tb-planning
Subject: Documentation fix patches

As I discuss various issues in bugs, I continually notice the lack of
documentation in so much of our internal code (Mozilla culture in
general does not support good code documentation per my standards, but
that is another issue. New code is routinely landed with no description
of the purpose of files or modules, for example).

Might I propose a new review rule, applying at least for anything under
c-c/mailnews and perhaps under other directories? That would be to allow
anyone who is a module owner or peer under c-c to provide a
documentation-only patch without review, landed with DONTBUILD? That way
we could reduce the friction in improving documentation. I actually
think there is a similar rule in existence in m-c  Does anyone know that
rule and what is entered for r= or a= to allow it?

:rkent

_______________________________________________
tb-planning mailing list
tb-planning at mozilla.org
https://mail.mozilla.org/listinfo/tb-planning

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.mozilla.org/pipermail/tb-planning/attachments/20160610/0d413800/attachment.html>


More information about the tb-planning mailing list