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_reviews&type=revision&diff=2163333&oldid=1910448

-- 

Andre Klapper | Wikimedia Bugwrangler
http://blogs.gnome.org/aklapper/

_______________________________________________
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Reply via email to