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
