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