[05:39:02] 10Phabricator: Private tasks should have a placeholder on the workboard - https://phabricator.wikimedia.org/T90833#2093494 (10Christopher) [11:55:54] 5Gerrit-Migration, 7Documentation: Update Code Review related documentation on wiki pages - https://phabricator.wikimedia.org/T207#2094023 (10Aklapper) Tempted to merge this into {T129067} and {T129068} - anyone disagreeing? [13:01:22] 5Gerrit-Migration, 10Analytics-Tech-community-metrics, 6Developer-Relations, 3DevRel-March-2016: Make MetricsGrimoire/korma support gathering Code Review statistics from Phabricator's Differential - https://phabricator.wikimedia.org/T118753#2094187 (10Aklapper) [13:36:37] 5Gerrit-Migration, 10Analytics-Tech-community-metrics, 6Developer-Relations, 3DevRel-March-2016: Make MetricsGrimoire/korma support gathering Code Review statistics from Phabricator's Differential - https://phabricator.wikimedia.org/T118753#2094217 (10Lcanasdiaz) @Aklapper any news about when you guys are... [16:26:07] 5Gerrit-Migration, 7Documentation: Update Code Review related documentation on wiki pages - https://phabricator.wikimedia.org/T207#2094690 (10greg) Well, as long as all of the crazy stuff under Gerrit that isn't (or shouldn't be) Gerrit specific is cleaned up and moved somewhere else (like under the proposed [... [17:22:29] 10Differential, 5Gerrit-Migration, 10Continuous-Integration-Infrastructure, 3releng-201516-q2, 3releng-201516-q3: [keyresult] Connect Differential code review with continuous integration - https://phabricator.wikimedia.org/T31#2095030 (10mmodell) [18:42:40] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2095410 (10mmodell) >>! In T121751#2093243, @Luke081515 wrote: > But not the commit content, for example if a user made a little mistake :-/... [18:48:37] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2095467 (10Dereckson) Actually, we have been using this workflow since 2012. This is the current Wikimedia practice to amend changes without... [19:05:58] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2095639 (10greg) >>! In T121751#2095467, @Dereckson wrote: > Actually, we have been using this workflow since 2012. That, in and of itself,... [19:07:22] 10Phabricator, 13Patch-For-Review, 7Regression: https://phabricator.wikimedia.org/maniphest/task/create/ 404 - https://phabricator.wikimedia.org/T127286#2095642 (10Dzahn) 5Open>3Resolved [19:08:53] 10Phabricator, 13Patch-For-Review, 7Regression: https://phabricator.wikimedia.org/maniphest/task/create/ 404 - https://phabricator.wikimedia.org/T127286#2038255 (10Dzahn) works, https://phabricator.wikimedia.org/maniphest/task/create not a 404 anymore [19:11:28] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2095668 (10Dereckson) Yes, I concur, we should evaluate the cost/benefit of the collaborative commit crafting model //per se//, and not only... [19:12:51] 5Gerrit-Migration, 7Documentation: Update Code Review related documentation on wiki pages from Gerrit to Differential - https://phabricator.wikimedia.org/T207#2095685 (10Aklapper) [19:13:26] 5Gerrit-Migration, 7Documentation: Update Code Review related documentation on wiki pages from Gerrit to Differential - https://phabricator.wikimedia.org/T207#2776 (10Aklapper) Makes a lot of sense. I clarified the task description and will keep this separate for the time being. [19:14:41] 5Gerrit-Migration, 7Documentation: Update Code Review related documentation on wiki pages from Gerrit to Differential - https://phabricator.wikimedia.org/T207#2095700 (10greg) Much better title, thanks :) [19:26:42] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2095761 (10mmodell) If we were modeling the wiki way, I would think that post-commit review would make more sense. But pre-commit review, whi... [19:30:46] 10Differential, 5Gerrit-Migration, 7Documentation, 7WorkType-NewFunctionality: Document gerrit workflows and demonstrate equivalent in arcanist workflows - https://phabricator.wikimedia.org/T117058#1765720 (10Dereckson) Could we also document on to easily use Arcanist to update three successive change? **... [20:18:20] 10Phabricator: Diffusion blob viewer may not work as expected - https://phabricator.wikimedia.org/T129129#2096059 (10Krinkle) [20:32:58] This will be super cool when upstream gets around to it: https://secure.phabricator.com/T9515 [20:35:50] whoa, holy redesign batman [20:36:29] I need to do more upstream work so I see those thigns faster :) [20:41:21] 5Gerrit-Migration, 7Documentation: Update Commit Message Guidelines for phab - https://phabricator.wikimedia.org/T123081#1920573 (10mmodell) I like the Test Plan field. I'm not so sure if it should be required though. Generally for code that is getting deployed to production, I think even having a revert plan... [22:25:37] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2097091 (10hashar) I constantly amend/finish up other authors changes. Either because I know the author is not going to respond any time soon... [22:27:52] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#1887024 (10Dzahn) >>! In T121751#1910101, @mmodell wrote: > Taking over someone's change is a rather heavy-handed thing to do so I don't thin... [22:31:13] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2097103 (10mmodell) >>! In T121751#2097093, @Dzahn wrote: >>>! In T121751#1910101, @mmodell wrote: >> Taking over someone's change is a rathe... [22:40:44] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2097164 (10mmodell) >>! In T121751#2097091, @hashar wrote: > I constantly amend/finish up other authors changes. Either because I know the au... [23:10:57] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2097325 (10hashar) @mmodell straight forward :-} That address **my** concern! [23:18:02] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2097361 (10greg) >>! In T121751#2097164, @mmodell wrote: >>>! In T121751#2097091, @hashar wrote: >> I constantly amend/finish up other author... [23:21:56] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2097376 (10greg) >>! In T121751#2097103, @mmodell wrote: >>>! In T121751#2097093, @Dzahn wrote: >>>>! In T121751#1910101, @mmodell wrote: >>>... [23:23:11] 10Differential, 5Gerrit-Migration, 7Documentation: Document best practices to amend a change written by another contributor - https://phabricator.wikimedia.org/T121751#2097379 (10greg) With these things documented, is this now resolvable? https://www.mediawiki.org/wiki/Phabricator/Differential [23:33:14] 10Differential, 5Gerrit-Migration, 7Documentation, 7WorkType-NewFunctionality: Document gerrit workflows and demonstrate equivalent in arcanist workflows - https://phabricator.wikimedia.org/T117058#2097407 (10mmodell) >>! In T117058#2095777, @Dereckson wrote: > Could we also document on to easily use Arcan... [23:34:41] 10Differential, 5Gerrit-Migration, 7Documentation, 7WorkType-NewFunctionality: Document gerrit workflows and demonstrate equivalent in arcanist workflows - https://phabricator.wikimedia.org/T117058#2097409 (10mmodell) >>! In T117058#2095777, @Dereckson wrote: > Could we also document on to easily use Arcan...