Change Details

//This task item was mentioned as one topic in the Developer Summit session T114419 and hence is a subtask of T101686.// **//Task summary to be improved - please edit.//** Improve initial quality of contributed patchsets and communicate expectations. **Actions:** * Clean up / rewrite https://www.mediawiki.org/wiki/Gerrit/Code_review/Getting_reviews (e.g. contains "Gerrit limitations" section which is irrelevant for a patch author); keep as generic / infrastructure-independent as possible (Gerrit vs Differential) * Incorporate author-related items from https://www.mediawiki.org/wiki/Gerrit/Code_review if appropriate * Check https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines * Potentially incorporate {T121751}? **Intention:** Improve the situation described in https://www.mediawiki.org/wiki/User:AKlapper_%28WMF%29/Code_Review#Poor_quality_of_contributors.27_patches (and to some extend https://www.mediawiki.org/wiki/User:AKlapper_%28WMF%29/Code_Review#Workload_of_existing_reviewers )

//This task item was mentioned as one topic in the Developer Summit session T114419 and hence is a subtask of T101686.// **//Task summary to be improved - please edit.//** Improve initial quality of contributed patchsets and communicate expectations. **Actions:** * Clean up / rewrite https://www.mediawiki.org/wiki/Gerrit/Code_review/Getting_reviews (e.g. contains "Gerrit limitations" section which is irrelevant for a patch author); keep as generic / infrastructure-independent as possible (Gerrit vs Differential) * Incorporate author-related items from https://www.mediawiki.org/wiki/Gerrit/Code_review if appropriate * Check https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines (and keep generic if possible - updating the page once Differential has replaced Gerrit is {T123081}) * Potentially incorporate {T121751}? **Intention:** Improve the situation described in https://www.mediawiki.org/wiki/User:AKlapper_%28WMF%29/Code_Review#Poor_quality_of_contributors.27_patches (and to some extend https://www.mediawiki.org/wiki/User:AKlapper_%28WMF%29/Code_Review#Workload_of_existing_reviewers )

//This task item was mentioned as one topic in the Developer Summit session T114419 and hence is a subtask of T101686.// **//Task summary to be improved - please edit.//** Improve initial quality of contributed patchsets and communicate expectations. **Actions:** * Clean up / rewrite https://www.mediawiki.org/wiki/Gerrit/Code_review/Getting_reviews (e.g. contains "Gerrit limitations" section which is irrelevant for a patch author); keep as generic / infrastructure-independent as possible (Gerrit vs Differential) * Incorporate author-related items from https://www.mediawiki.org/wiki/Gerrit/Code_review if appropriate * Check https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines (and keep generic if possible - updating the page once Differential has replaced Gerrit is {T123081}) * Potentially incorporate {T121751}? **Intention:** Improve the situation described in https://www.mediawiki.org/wiki/User:AKlapper_%28WMF%29/Code_Review#Poor_quality_of_contributors.27_patches (and to some extend https://www.mediawiki.org/wiki/User:AKlapper_%28WMF%29/Code_Review#Workload_of_existing_reviewers )