14:28:49 <karsten> #startmeeting metrics team 14:28:49 <MeetBot> Meeting started Thu Jul 5 14:28:49 2018 UTC. The chair is karsten. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:28:49 <MeetBot> Useful Commands: #action #agreed #help #info #idea #link #topic. 14:28:53 <karsten> hi irl! 14:28:56 <irl> hi! 14:29:00 <karsten> https://storm.torproject.org/shared/5h1Goax5eNusxjXJ_Ty5Wl7hFR1uqCReUiN8xdlBG8T <- agenda pad 14:29:21 <karsten> looks like we have a single topic for today: 14:29:26 <karsten> * GitHub (irl) 14:29:39 <irl> this is from isabela's email 14:29:52 <karsten> yep. makes sense to discuss this here before replying. 14:30:16 <irl> i don't believe it hurts to have mirrors of things on github, and if we have a bot set up for this then that's even better 14:30:28 <karsten> agreed. 14:30:40 <karsten> I think we're less affected than other teams, 14:30:45 <irl> we have in the past had trac tickets with github branches for relay search and that was useful 14:31:01 <irl> the contributors didn't really hang around long enough that it would have made sense to give them git.tpo accounts 14:31:06 <karsten> true. 14:31:25 <irl> shall i follow up then with "we would like mirrors of things on github"? 14:31:27 <karsten> I'm also curious about using it for longer reviews. 14:31:42 <irl> i guess you get the ability for inline comments on commits 14:31:43 <karsten> in the past, iwakeh had really long trac tickets for reviews. 14:31:49 <karsten> iwakeh and I* 14:32:25 <karsten> yes, please follow up. what else do you need to know? 14:32:37 <irl> i think that's all 14:32:48 <karsten> my github name is kloesing. in case it matters. 14:32:57 <irl> ah yes, i did need that 14:33:07 <irl> but then i think that's all 14:33:14 <karsten> great! sounds good. 14:33:50 <karsten> is there anything else we should discuss? anything you need from me? 14:34:39 <irl> for the rest of the week i plan to look at relay search tickets and see which i can close out easily, and will just pick up reviews when they appear 14:34:57 <irl> unless there is anything i should prioritise? 14:35:07 <karsten> nope. sounds like a fine plan. 14:35:30 <karsten> do you need me to review something there? 14:36:00 <irl> i think everything i've done so far is already needs_review 14:36:13 <irl> the important one was the integration of relay search so that smaller patches could follow 14:36:29 <irl> the relay search patches probably do not need any detailed review 14:36:39 <irl> in the past i've committed directly to master 14:37:00 <karsten> https://trac.torproject.org/projects/tor/query?status=needs_review&component=%5EMetrics&col=id&col=summary&col=component&col=status&col=type&col=priority&col=milestone&col=reviewer&order=priority 14:38:18 <karsten> how should we commit RS-related things now? 14:38:30 <karsten> should I keep doing that? 14:38:48 <karsten> well, by keep doing I mean should I continue being the person pushing to metrics-web? 14:39:08 <karsten> because I'll also have to deploy afterwards. (at some point we'll need to set you up as co-maintainer of metrics-web.) 14:39:34 <irl> probably you don't want to merge individual tickets, i can instead have a relaysearch-dev branch that i queue things up on and you can just merge that branch 14:39:53 <karsten> okay, that would work. 14:40:21 <irl> let's try it out for a while and see how it goes 14:40:26 <karsten> yep! 14:40:50 <irl> the only other question i have is the admin question, of which sponsor my time goes against 14:41:05 <karsten> I'll find out after this meeting. I hope. 14:41:11 <karsten> and then let you know. 14:41:13 <irl> ok cool, i'll look out for an email (: 14:41:17 <karsten> heh 14:41:32 <irl> ok, that's all from me, no more topics 14:41:54 <karsten> great! also, no more topics from me. 14:42:24 <karsten> thanks, and let's continue conversations on trac. bye! :) 14:42:39 <karsten> #endmeeting