As part of the effort to improve our code review process, the
documentation for patch *authors* saw some improvements at
https://www.mediawiki.org/wiki/Gerrit/Code_review/Getting_reviews
The page now lists some prerequisites, expectations about the patch
(scope, testing, docs), and a new section about dealing with obstacles
(no timely review, further reasons for rework or rejection). [1],[2]
Your feedback is welcome on its talk/discussion page!
The next step will be to discuss having a structured, standardized
approach for reviewing code contributions and to generally improve the
documentation for reviewers.
Please help in
https://phabricator.wikimedia.org/T129067
Cheers,
andre
[1]
https://phabricator.wikimedia.org/T129068
[2] Diff:
https://www.mediawiki.org/w/index.php?title=Gerrit%2FCode_review%2FGetting_…
--
Andre Klapper | Wikimedia Bugwrangler
http://blogs.gnome.org/aklapper/