{
  "founder": "wumpus",
  "channel": "#bitcoin-core-dev",
  "network": "freenode",
  "id": "62a73d028b60465ab56200334d6b2bcd",
  "name": "#bitcoin-core-dev",
  "chair": "wumpus",
  "chairs": [
    "wumpus"
  ],
  "nicks": {
    "wumpus": 86,
    "lightningbot": 2,
    "cfields": 21,
    "sipa": 46,
    "CodeShark": 4,
    "achow101": 21,
    "Provoostenator": 19,
    "instagibbs": 17,
    "jonasschnelli": 50,
    "meshcollider": 5,
    "luke-jr": 15,
    "BlueMatt": 29,
    "jtimon": 28,
    "karelb": 4,
    "gribble": 10,
    "gmaxwell": 42,
    "Randolf": 1,
    "jnewbery": 2,
    "morcos": 4,
    "kanzure": 2,
    "MarcoFalke": 10,
    "ryanofsky": 1,
    "aj": 2
  },
  "start_time": "2017-11-30T19:00:03+00:00",
  "end_time": "2017-11-30T20:00:33+00:00",
  "active": false,
  "original_topic": "Bitcoin Core development discussion and commit log | This is the channel for developing Bitcoin Core. Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: https://botbot.me/freenode/bitcoin-core-dev, http://www.erisian.com.au/bitcoin-core-dev/",
  "current_topic": "review nit reduction",
  "messages": [
    {
      "id": "5a5587551f644c148a179f39ce9ceb49",
      "sender": "wumpus",
      "payload": "#startmeeting",
      "action": false,
      "timestamp": "2017-11-30T19:00:03+00:00"
    },
    {
      "id": "4b542e58bebf4fc083edf61d5e9897eb",
      "sender": "lightningbot",
      "payload": "Meeting started Thu Nov 30 19:00:03 2017 UTC.  The chair is wumpus. Information about MeetBot at http://wiki.debian.org/MeetBot.",
      "action": false,
      "timestamp": "2017-11-30T19:00:03+00:00"
    },
    {
      "id": "5fe05e771d0948e4a749d4d0fa6839a8",
      "sender": "lightningbot",
      "payload": "Useful Commands: #action #agreed #help #info #idea #link #topic.",
      "action": false,
      "timestamp": "2017-11-30T19:00:03+00:00"
    },
    {
      "id": "0dbde8e0ccf44682af0be36dd4521df5",
      "sender": "wumpus",
      "payload": "#bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012 achow101 meshcollider jnewbery maaku fanquake promag",
      "action": false,
      "timestamp": "2017-11-30T19:00:23+00:00"
    },
    {
      "id": "0c0026d603c348dfb278b86a554a9111",
      "sender": "cfields",
      "payload": "hi",
      "action": false,
      "timestamp": "2017-11-30T19:00:32+00:00"
    },
    {
      "id": "fc01990db9224a17a547eb5665145a30",
      "sender": "sipa",
      "payload": "hi",
      "action": false,
      "timestamp": "2017-11-30T19:00:35+00:00"
    },
    {
      "id": "b7c360c1f0684d638619c81c8f236c62",
      "sender": "CodeShark",
      "payload": "hello",
      "action": false,
      "timestamp": "2017-11-30T19:00:36+00:00"
    },
    {
      "id": "b1529b53d51c49fdad4647eea7e99ee9",
      "sender": "achow101",
      "payload": "hi",
      "action": false,
      "timestamp": "2017-11-30T19:00:44+00:00"
    },
    {
      "id": "137451d29d554344b520c72152e1d547",
      "sender": "Provoostenator",
      "payload": "hi",
      "action": false,
      "timestamp": "2017-11-30T19:00:46+00:00"
    },
    {
      "id": "87d8bf06ccd5417abd7303b6df6380c1",
      "sender": "instagibbs",
      "payload": "hi",
      "action": false,
      "timestamp": "2017-11-30T19:00:51+00:00"
    },
    {
      "id": "c6caefca72b54c7ca2c9b1f03d16551c",
      "sender": "jonasschnelli",
      "payload": "hi",
      "action": false,
      "timestamp": "2017-11-30T19:00:55+00:00"
    },
    {
      "id": "bdf4370f984040b2b6926110944ba5d0",
      "sender": "wumpus",
      "payload": "#topic high priority for review",
      "action": false,
      "timestamp": "2017-11-30T19:00:59+00:00"
    },
    {
      "id": "8470230ba750449c9e7658e3e66f4c68",
      "sender": "meshcollider",
      "payload": "hi",
      "action": false,
      "timestamp": "2017-11-30T19:01:01+00:00"
    },
    {
      "id": "947c826e554740fe97e4644d77001546",
      "sender": "luke-jr",
      "payload": "hi",
      "action": false,
      "timestamp": "2017-11-30T19:01:16+00:00"
    },
    {
      "id": "0a86d28c38944ec096ff4801eefce537",
      "sender": "wumpus",
      "payload": "8 PRs have been tagged for https://github.com/bitcoin/bitcoin/projects/8",
      "action": false,
      "timestamp": "2017-11-30T19:01:20+00:00"
    },
    {
      "id": "1049dd935cfb4977a6dbb11b6f2a0de1",
      "sender": "BlueMatt",
      "payload": "why does cfields have 2?",
      "action": false,
      "timestamp": "2017-11-30T19:01:39+00:00"
    },
    {
      "id": "2cb5139cbf9f4ca49c9acfa0a3fa038e",
      "sender": "wumpus",
      "payload": "I added cfields's BanMan last week as it's blocking his further net refactoring",
      "action": false,
      "timestamp": "2017-11-30T19:01:44+00:00"
    },
    {
      "id": "2b91edc55bf24926a96c30161160aa02",
      "sender": "cfields",
      "payload": "BlueMatt: wasn't me, but i'll take it :)",
      "action": false,
      "timestamp": "2017-11-30T19:01:59+00:00"
    },
    {
      "id": "1396004f4e844f62b1d16b7d77a1b2cf",
      "sender": "wumpus",
      "payload": "ohh is it unfair?",
      "action": false,
      "timestamp": "2017-11-30T19:02:06+00:00"
    },
    {
      "id": "7996fa9ef8464fc9b9eefe6b119a68fa",
      "sender": "jonasschnelli",
      "payload": "also have two :|",
      "action": true,
      "timestamp": "2017-11-30T19:02:09+00:00"
    },
    {
      "id": "159c22c9ff764122843498b1440f799a",
      "sender": "instagibbs",
      "payload": "achow101, i see you rebased yours? we shooting for post-0.16?",
      "action": false,
      "timestamp": "2017-11-30T19:02:18+00:00"
    },
    {
      "id": "d805582d6eaa414ea18e544d34469b13",
      "sender": "BlueMatt",
      "payload": "i think we try to have only one per person",
      "action": false,
      "timestamp": "2017-11-30T19:02:21+00:00"
    },
    {
      "id": "8244484f63ed4780bf5d862fa52d7915",
      "sender": "jtimon",
      "payload": "hi",
      "action": false,
      "timestamp": "2017-11-30T19:02:24+00:00"
    },
    {
      "id": "6236938482dc43b2a6898d709e3f6683",
      "sender": "BlueMatt",
      "payload": "well i suggested we include jonasschnelli's other one as its kinda a segwit-wallet-blocker imo",
      "action": false,
      "timestamp": "2017-11-30T19:02:34+00:00"
    },
    {
      "id": "d2b719d363fd4f88ad15bf5920b4eff9",
      "sender": "BlueMatt",
      "payload": "or at least in-same-release",
      "action": false,
      "timestamp": "2017-11-30T19:02:38+00:00"
    },
    {
      "id": "bfc03c7a304b49e8ac0a43db1de2df64",
      "sender": "achow101",
      "payload": "instagibbs: I'm shooting for 0.16",
      "action": false,
      "timestamp": "2017-11-30T19:02:42+00:00"
    },
    {
      "id": "4bdafc41a3dc4e68b58cc26e62a4b4b0",
      "sender": "achow101",
      "payload": "but it may not make it",
      "action": false,
      "timestamp": "2017-11-30T19:02:47+00:00"
    },
    {
      "id": "4d495d826a274733ae1cf14652de4cad",
      "sender": "instagibbs",
      "payload": "ok, I can give it more love, though I think I've given a lot already, probably needs others....",
      "action": false,
      "timestamp": "2017-11-30T19:03:04+00:00"
    },
    {
      "id": "ca8517e7d3004d76aeb4ee8103be9764",
      "sender": "achow101",
      "payload": "still some segwit related things to do and runn1ng wants simulations",
      "action": false,
      "timestamp": "2017-11-30T19:03:21+00:00"
    },
    {
      "id": "8516f4a5ef284951b70f21285c97354e",
      "sender": "achow101",
      "payload": "also been busy with exams and classwork",
      "action": false,
      "timestamp": "2017-11-30T19:03:31+00:00"
    },
    {
      "id": "b8d4e84195e14027982f8a1abf928987",
      "sender": "wumpus",
      "payload": "well, good solution for that, review and merge one of cfields's PRs and there will be only one left :)",
      "action": false,
      "timestamp": "2017-11-30T19:03:37+00:00"
    },
    {
      "id": "6e2d3b17e5e3428db0f4e18b2c0d9b87",
      "sender": "karelb",
      "payload": "achow101: I came just in time :) yes I wanted some simulations",
      "action": false,
      "timestamp": "2017-11-30T19:03:44+00:00"
    },
    {
      "id": "ae2fe31495314bdf958da4652965feb1",
      "sender": "jonasschnelli",
      "payload": "wumpus: heh",
      "action": false,
      "timestamp": "2017-11-30T19:03:51+00:00"
    },
    {
      "id": "15b84e3b6342430fa13dfb7316aea7a0",
      "sender": "sipa",
      "payload": "so i think we're maybe unintentionally moving from a \"we release regularly with whatever is finished\" to \"X is a blocker for release Y\"",
      "action": false,
      "timestamp": "2017-11-30T19:03:57+00:00"
    },
    {
      "id": "3765481de46241dc94eae87f64313887",
      "sender": "BlueMatt",
      "payload": "oh, wait, i dont have one...hmmmm, I'd say #10279",
      "action": false,
      "timestamp": "2017-11-30T19:04:10+00:00"
    },
    {
      "id": "7eb639844e084727b2630a7526a51cf7",
      "sender": "wumpus",
      "payload": "sipa: segwit is the only exception to that",
      "action": false,
      "timestamp": "2017-11-30T19:04:13+00:00"
    },
    {
      "id": "80690c0bb99445c998acc9ac6662e8bf",
      "sender": "cfields",
      "payload": "i have no problem with removing one of mine if it's an issue. But yes, I think 11363 is ready/easy to review",
      "action": false,
      "timestamp": "2017-11-30T19:04:14+00:00"
    },
    {
      "id": "523c93e357ff429ebd26c853874e2780",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/10279 | Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces by TheBlueMatt \u00c3\u0082\u00c2\u00b7 Pull Request #10279 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:04:15+00:00"
    },
    {
      "id": "f4812b55e9b34d0fa0b822b4dabb8d4a",
      "sender": "BlueMatt",
      "payload": "sipa: I think we'll go back to that post-segwit-wallet, no?",
      "action": false,
      "timestamp": "2017-11-30T19:04:23+00:00"
    },
    {
      "id": "2708965ec0024480a4654a8c48dbcda3",
      "sender": "wumpus",
      "payload": "sipa: for the rest, releases are only ever blocked on bugfixes not features",
      "action": false,
      "timestamp": "2017-11-30T19:04:25+00:00"
    },
    {
      "id": "01d088073c2c479981f9cb48a8664957",
      "sender": "achow101",
      "payload": "sipa: I think 0.16 is the exception? because segwit wallet",
      "action": false,
      "timestamp": "2017-11-30T19:04:29+00:00"
    },
    {
      "id": "48892dc15cf74de1857df905276821a5",
      "sender": "wumpus",
      "payload": "yes",
      "action": false,
      "timestamp": "2017-11-30T19:04:35+00:00"
    },
    {
      "id": "1e6d252bc2014604938ccac452e6efd7",
      "sender": "jtimon",
      "payload": "sipa: what is the blocker for 0.16 ?",
      "action": false,
      "timestamp": "2017-11-30T19:04:36+00:00"
    },
    {
      "id": "89e346cfa7804f3a92abb2190e4055d2",
      "sender": "sipa",
      "payload": "okay, that's fine - i'm not saying it's a bad thing",
      "action": false,
      "timestamp": "2017-11-30T19:04:41+00:00"
    },
    {
      "id": "1200c5712efc4528abee241ed393f39e",
      "sender": "instagibbs",
      "payload": "i think it's openly acknowledged",
      "action": false,
      "timestamp": "2017-11-30T19:04:50+00:00"
    },
    {
      "id": "94dc3bd3a0b74748b625d3f41c28cd6c",
      "sender": "wumpus",
      "payload": "and segwit wallet is not intended as a *blocker* for 0.16",
      "action": false,
      "timestamp": "2017-11-30T19:04:55+00:00"
    },
    {
      "id": "ea15b5638b48476883f719753bb35531",
      "sender": "wumpus",
      "payload": "we intend to release 0.16 early after that's in",
      "action": false,
      "timestamp": "2017-11-30T19:05:07+00:00"
    },
    {
      "id": "f8d4c11e5eb64d59a6427362ce452704",
      "sender": "wumpus",
      "payload": "so it's more like a trigger",
      "action": false,
      "timestamp": "2017-11-30T19:05:13+00:00"
    },
    {
      "id": "4352509c44e141a5861a046d28dd13aa",
      "sender": "sipa",
      "payload": "fair enough",
      "action": false,
      "timestamp": "2017-11-30T19:05:13+00:00"
    },
    {
      "id": "bf884ec7702e42f0a81ce5bb5cba2e70",
      "sender": "sipa",
      "payload": "but it still changes prioritization a bit",
      "action": false,
      "timestamp": "2017-11-30T19:05:20+00:00"
    },
    {
      "id": "df0a545029ae4a8d925dc067652349da",
      "sender": "BlueMatt",
      "payload": "indeed",
      "action": false,
      "timestamp": "2017-11-30T19:05:28+00:00"
    },
    {
      "id": "daa25d2238a04a8ab7f520f926382458",
      "sender": "instagibbs",
      "payload": "i think the segwit pr is shaping up nicely at least",
      "action": false,
      "timestamp": "2017-11-30T19:05:32+00:00"
    },
    {
      "id": "cbc01b8474a043058e651054ea528ecb",
      "sender": "wumpus",
      "payload": "if segwit wallet takes longer than the 0.16 release window, then IMO we should release 0.16 without it",
      "action": false,
      "timestamp": "2017-11-30T19:05:34+00:00"
    },
    {
      "id": "d1f7f74cc7dc40288ae0be2178245450",
      "sender": "wumpus",
      "payload": "but I don't expect that",
      "action": false,
      "timestamp": "2017-11-30T19:05:43+00:00"
    },
    {
      "id": "930e64474c0445bba7011d83a6e3dfae",
      "sender": "BlueMatt",
      "payload": "wumpus: agreed",
      "action": false,
      "timestamp": "2017-11-30T19:05:48+00:00"
    },
    {
      "id": "36035d6c22804bc1918d057694455fbb",
      "sender": "jtimon",
      "payload": "oh, I see, we want to release 0.16 soon after segwit wallet gets in",
      "action": false,
      "timestamp": "2017-11-30T19:06:09+00:00"
    },
    {
      "id": "6dca1dcfdb4d41f1b099b7632e9ce932",
      "sender": "wumpus",
      "payload": "jtimon: yep",
      "action": false,
      "timestamp": "2017-11-30T19:06:14+00:00"
    },
    {
      "id": "7229d99c0cd84f65872f237a28ca4c8f",
      "sender": "sipa",
      "payload": "i'm about to push some review fixes to #11403",
      "action": false,
      "timestamp": "2017-11-30T19:06:26+00:00"
    },
    {
      "id": "058395c87c484650b3edea98443e02d4",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #11403 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:06:30+00:00"
    },
    {
      "id": "088b11a0ea1242e29e7168c480086c73",
      "sender": "wumpus",
      "payload": "jtimon: seems more advisable than trying to backport the whole lot + dependencies to 0.15",
      "action": false,
      "timestamp": "2017-11-30T19:06:31+00:00"
    },
    {
      "id": "ec55401d280c4279ae12ece1453416df",
      "sender": "cfields",
      "payload": "sipa: not sure if you've clarified somewhere, do you consider all of the listed TODOs in 11403 blockers for release as well?",
      "action": false,
      "timestamp": "2017-11-30T19:06:34+00:00"
    },
    {
      "id": "699fe9db34b24b7f8228916390dbdbf6",
      "sender": "jtimon",
      "payload": "wumpus: yeah, I think I suggested that, eithe way it makes sense to me",
      "action": false,
      "timestamp": "2017-11-30T19:06:53+00:00"
    },
    {
      "id": "96040c1dd0eb472bbb0352e27ec5ef95",
      "sender": "wumpus",
      "payload": "jtimon: thanks in that case :)",
      "action": false,
      "timestamp": "2017-11-30T19:07:18+00:00"
    },
    {
      "id": "947edb99315946c9a11790c0cbc4a62b",
      "sender": "luke-jr",
      "payload": "branch 0.15 into 0.16 ;)",
      "action": false,
      "timestamp": "2017-11-30T19:07:26+00:00"
    },
    {
      "id": "7ff0c259aec344219bc42532ba4af9ec",
      "sender": "sipa",
      "payload": "cfields: yes",
      "action": false,
      "timestamp": "2017-11-30T19:07:32+00:00"
    },
    {
      "id": "524d1040d70743f2ba2e51c1bf8a2291",
      "sender": "cfields",
      "payload": "ok, thanks",
      "action": false,
      "timestamp": "2017-11-30T19:07:48+00:00"
    },
    {
      "id": "544ef1901ad4421a831cf454ccfbb15a",
      "sender": "gmaxwell",
      "payload": "So, segwit-wallet is done?",
      "action": false,
      "timestamp": "2017-11-30T19:08:03+00:00"
    },
    {
      "id": "15c39523fdfd46e9b5f9dc8e86c28f17",
      "sender": "jtimon",
      "payload": "I'm not following review on the segwit wallet pr, how is it going?",
      "action": false,
      "timestamp": "2017-11-30T19:08:15+00:00"
    },
    {
      "id": "f71ca4dbfcfa4c03accd6446a4893690",
      "sender": "sipa",
      "payload": "i think #11403 is pretty much done - just nits in command line option handling and output",
      "action": false,
      "timestamp": "2017-11-30T19:08:31+00:00"
    },
    {
      "id": "d696a13f82e747e1a4f175c6b1561ff9",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #11403 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:08:34+00:00"
    },
    {
      "id": "916760ef3ee6425482e1ad37fa0d18c1",
      "sender": "jtimon",
      "payload": "great",
      "action": false,
      "timestamp": "2017-11-30T19:08:45+00:00"
    },
    {
      "id": "be58081b73504d25ad9bc335af75369d",
      "sender": "achow101",
      "payload": "I tried reviewing it but it's big and scary",
      "action": false,
      "timestamp": "2017-11-30T19:08:55+00:00"
    },
    {
      "id": "d4d97e5ebee74643a2ab0414397b0885",
      "sender": "wumpus",
      "payload": "awesome",
      "action": false,
      "timestamp": "2017-11-30T19:09:10+00:00"
    },
    {
      "id": "be2e7c32413c4afab3d21092e26c4dc2",
      "sender": "sipa",
      "payload": "we may want to discuss what to do with things like how dumpprivkey and signmessage should work in a post-segwit world",
      "action": false,
      "timestamp": "2017-11-30T19:09:23+00:00"
    },
    {
      "id": "39e4685e9b9547a3a045df3a445bc423",
      "sender": "Randolf",
      "payload": "congratulates achow101 for trying",
      "action": true,
      "timestamp": "2017-11-30T19:09:29+00:00"
    },
    {
      "id": "040b62db7dbe410792862b1b07aab73b",
      "sender": "cfields",
      "payload": "i found it reasonable review on a per-commit basis",
      "action": false,
      "timestamp": "2017-11-30T19:09:30+00:00"
    },
    {
      "id": "fa5a4b6cb2fe46bcbd056c38b38ec83d",
      "sender": "instagibbs",
      "payload": "cfields, same",
      "action": false,
      "timestamp": "2017-11-30T19:09:37+00:00"
    },
    {
      "id": "96b192df40a14a4f99f1fed59aea164f",
      "sender": "sipa",
      "payload": "as i believe some projects have come up with formats on their own",
      "action": false,
      "timestamp": "2017-11-30T19:09:40+00:00"
    },
    {
      "id": "19487a027d5e472e8418d296c445d400",
      "sender": "cfields",
      "payload": "*to review",
      "action": false,
      "timestamp": "2017-11-30T19:09:41+00:00"
    },
    {
      "id": "2be87e32626b41bda90c32d173aab2c8",
      "sender": "achow101",
      "payload": "sipa: trezor has implemented their own segwit message signing thing",
      "action": false,
      "timestamp": "2017-11-30T19:09:55+00:00"
    },
    {
      "id": "7c91a631b8df4b2fb0e7390fb431a640",
      "sender": "jtimon",
      "payload": "I guess I need tofix the nits in #8994 and #10757 if I want them to have a chance to get merged before 0.16 then",
      "action": false,
      "timestamp": "2017-11-30T19:09:59+00:00"
    },
    {
      "id": "9cbcde1d9f74475eac03341e19c51df8",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon \u00c3\u0082\u00c2\u00b7 Pull Request #8994 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:10:02+00:00"
    },
    {
      "id": "2ecf04ce0ce44ee39472252dbd0e3403",
      "sender": "achow101",
      "payload": "I think that might be the only one though",
      "action": false,
      "timestamp": "2017-11-30T19:10:03+00:00"
    },
    {
      "id": "d616789536014e81af875e4a0b0661f5",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/10757 | RPC: Introduce getblockstats to plot things by jtimon \u00c3\u0082\u00c2\u00b7 Pull Request #10757 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:10:07+00:00"
    },
    {
      "id": "aa504413b8384f389d0d859b0a5dd0ea",
      "sender": "sipa",
      "payload": "achow101: perhaps we can just adopt that",
      "action": false,
      "timestamp": "2017-11-30T19:10:22+00:00"
    },
    {
      "id": "50a5b2de081f46759bbd364e4a7adec0",
      "sender": "jtimon",
      "payload": "what are we referring to by \"soon after\"?",
      "action": false,
      "timestamp": "2017-11-30T19:10:25+00:00"
    },
    {
      "id": "8a4a8611fcb047668c811e6e68bed2ba",
      "sender": "wumpus",
      "payload": "sipa: well if other project's import/export formats make sense, it'd be good to use them for interchangeability",
      "action": false,
      "timestamp": "2017-11-30T19:10:30+00:00"
    },
    {
      "id": "6032109682f140e09717afa59e2420c4",
      "sender": "achow101",
      "payload": "sipa: I don't think it's documented anywhere though",
      "action": false,
      "timestamp": "2017-11-30T19:10:45+00:00"
    },
    {
      "id": "88d871d93fcb42dca8006c2477ffc55f",
      "sender": "meshcollider",
      "payload": "Should there be a BIP to formalise it",
      "action": false,
      "timestamp": "2017-11-30T19:10:48+00:00"
    },
    {
      "id": "ea013c84296547d680c4b056fb609bba",
      "sender": "achow101",
      "payload": "at least not yet",
      "action": false,
      "timestamp": "2017-11-30T19:10:48+00:00"
    },
    {
      "id": "e4577c40d7234041bd6ddf3c5ff94ecf",
      "sender": "Provoostenator",
      "payload": "Is there a good standard for message signing?",
      "action": false,
      "timestamp": "2017-11-30T19:10:52+00:00"
    },
    {
      "id": "78a47674549c46348b86edee1584f791",
      "sender": "sipa",
      "payload": "longer term i think a script-based signing would nice, but i don't think we're ready to adopt something like that",
      "action": false,
      "timestamp": "2017-11-30T19:10:53+00:00"
    },
    {
      "id": "e25ec4d7c606469187c6c1653eeaa8d9",
      "sender": "gmaxwell",
      "payload": "I thought they did but backed it out because it wasn't standardized yet. What they did didn't seem to awful, though I feel a little uneasy about the mutability between keytypes.",
      "action": false,
      "timestamp": "2017-11-30T19:10:59+00:00"
    },
    {
      "id": "9225703b14a54ec79e7b86dba8b65bf2",
      "sender": "karelb",
      "payload": "sipa: ad sign message - in trezor we changed v constant - https://github.com/bitcoin/bitcoin/issues/10542#issuecomment-316032523 - no BIP, no time :(",
      "action": false,
      "timestamp": "2017-11-30T19:11:00+00:00"
    },
    {
      "id": "79e3b6484a95417eac1eb63b271eee49",
      "sender": "sipa",
      "payload": "karelb: ok, i'll have a look",
      "action": false,
      "timestamp": "2017-11-30T19:11:13+00:00"
    },
    {
      "id": "ec2fdbf0c2ba4938881fa297a80fedfc",
      "sender": "sipa",
      "payload": "the original bitcoin core message signing doesn't have a bip either",
      "action": false,
      "timestamp": "2017-11-30T19:11:22+00:00"
    },
    {
      "id": "e3f3f54536914b10a606f33c31d3a281",
      "sender": "luke-jr",
      "payload": "Provoostenator: no",
      "action": false,
      "timestamp": "2017-11-30T19:11:33+00:00"
    },
    {
      "id": "f68c6ecf69db43388bcf672a32fd1612",
      "sender": "wumpus",
      "payload": "a BIP is not a requirement for us implementing it, though in parallel it'd be nice",
      "action": false,
      "timestamp": "2017-11-30T19:11:40+00:00"
    },
    {
      "id": "8a69f4bd280b430cb9548bbbe5d2cbdf",
      "sender": "BlueMatt",
      "payload": "sipa: it probably should if we change it, though",
      "action": false,
      "timestamp": "2017-11-30T19:11:43+00:00"
    },
    {
      "id": "db880b1ef46c44d897c7fbb33950cb5c",
      "sender": "achow101",
      "payload": "sipa: perhaps it should? retcon one :p",
      "action": false,
      "timestamp": "2017-11-30T19:11:45+00:00"
    },
    {
      "id": "ccff70c388044c61b044ab790c5d84bc",
      "sender": "sipa",
      "payload": "seems reasonable",
      "action": false,
      "timestamp": "2017-11-30T19:11:56+00:00"
    },
    {
      "id": "138c3aec0bc749868d51be1b1e217a70",
      "sender": "wumpus",
      "payload": "good to already have implementations and the BIP just formalizes it",
      "action": false,
      "timestamp": "2017-11-30T19:11:57+00:00"
    },
    {
      "id": "cb59ba5d69424d6e8b5df4da0f4031a0",
      "sender": "gmaxwell",
      "payload": "Well, signmessage is generally not very good; but fixing it should be a longer term thing.",
      "action": false,
      "timestamp": "2017-11-30T19:12:08+00:00"
    },
    {
      "id": "20c2a10eed6f453c9fc794b62f39558e",
      "sender": "Provoostenator",
      "payload": "I vaguely remember trying to implement / refactor signing and being not amused by the lack of a standard.",
      "action": false,
      "timestamp": "2017-11-30T19:12:09+00:00"
    },
    {
      "id": "37cf5e749c8b4957b8b11c5059d44447",
      "sender": "luke-jr",
      "payload": "current signed messages get very little real use, and most \"usage\" is based on misconceptions",
      "action": false,
      "timestamp": "2017-11-30T19:12:16+00:00"
    },
    {
      "id": "32cb113b5fb14661bc27f294c0f3d50c",
      "sender": "karelb",
      "payload": "sipa yes good point. I wanted to write up a document that describes both current and segwit message signing, but I got stuck on how ecdsa signing actually works :)",
      "action": false,
      "timestamp": "2017-11-30T19:12:23+00:00"
    },
    {
      "id": "59009487a63d48b79546b0d0e53c45b5",
      "sender": "wumpus",
      "payload": "luke-jr: agree",
      "action": false,
      "timestamp": "2017-11-30T19:12:24+00:00"
    },
    {
      "id": "dbedd800172b4badbcf54d67ab267c3a",
      "sender": "luke-jr",
      "payload": "IMO it'd make sense to just deprecate it until a better replacement is made",
      "action": false,
      "timestamp": "2017-11-30T19:12:26+00:00"
    },
    {
      "id": "f49bbdd2156142a68cb55d727b4ac8c9",
      "sender": "wumpus",
      "payload": "luke-jr: it's not a priority at least",
      "action": false,
      "timestamp": "2017-11-30T19:12:29+00:00"
    },
    {
      "id": "89571dd868e94f63886c40606d3f8048",
      "sender": "instagibbs",
      "payload": "luke-jr, it's used for airdrops :P",
      "action": false,
      "timestamp": "2017-11-30T19:12:33+00:00"
    },
    {
      "id": "73a7ca50ac5546b796b87fc5323dac6c",
      "sender": "sipa",
      "payload": "luke-jr: i agree it's not all that useful",
      "action": false,
      "timestamp": "2017-11-30T19:12:40+00:00"
    },
    {
      "id": "7824f0c9418d4776a9a8e14f6864f472",
      "sender": "Provoostenator",
      "payload": "Can it be made coin-agnostic as well?",
      "action": false,
      "timestamp": "2017-11-30T19:12:44+00:00"
    },
    {
      "id": "8683ff7f92e6491fac44bee80d8ced26",
      "sender": "wumpus",
      "payload": "certainly shouldn't be a blocker for 0.16",
      "action": false,
      "timestamp": "2017-11-30T19:12:48+00:00"
    },
    {
      "id": "307f11f0fd08408bb27efd89c6013a01",
      "sender": "luke-jr",
      "payload": "instagibbs: that's a misuse, since it doesn't prove you have funds ;)",
      "action": false,
      "timestamp": "2017-11-30T19:12:50+00:00"
    },
    {
      "id": "748ed3c327d7492f8299795f07840ce5",
      "sender": "gmaxwell",
      "payload": "Provoostenator: no.",
      "action": false,
      "timestamp": "2017-11-30T19:12:51+00:00"
    },
    {
      "id": "585a85d461684af1bed9f79fd65f25e1",
      "sender": "karelb",
      "payload": "we didn't have segwit message signing in web wallet, but users repeatedly demanded it",
      "action": false,
      "timestamp": "2017-11-30T19:13:09+00:00"
    },
    {
      "id": "923c1e16a05944928f0692e3f6c5c2b4",
      "sender": "instagibbs",
      "payload": "luke-jr, but i profit from it, how can it be misuse ;P",
      "action": false,
      "timestamp": "2017-11-30T19:13:12+00:00"
    },
    {
      "id": "22a298f226984205a85935e34ac8ae3e",
      "sender": "instagibbs",
      "payload": "+1 tho",
      "action": false,
      "timestamp": "2017-11-30T19:13:19+00:00"
    },
    {
      "id": "cb9d858ce3c646299795f1e5022f2e55",
      "sender": "gmaxwell",
      "payload": "The fact that it's inherently incompatible with multisig is a real bummer.",
      "action": false,
      "timestamp": "2017-11-30T19:13:21+00:00"
    },
    {
      "id": "ae87a35e120a4ed8b296c87a33e8a342",
      "sender": "luke-jr",
      "payload": "karelb: for what, though?",
      "action": false,
      "timestamp": "2017-11-30T19:13:22+00:00"
    },
    {
      "id": "f133dd1c8e284e9fb3001cc7d7120337",
      "sender": "achow101",
      "payload": "instagibbs: lol",
      "action": false,
      "timestamp": "2017-11-30T19:13:27+00:00"
    },
    {
      "id": "a7c44b99d2af4a54ba59e483a975fde9",
      "sender": "gmaxwell",
      "payload": "luke-jr: airdrops",
      "action": false,
      "timestamp": "2017-11-30T19:13:31+00:00"
    },
    {
      "id": "c37d06df479940bdb4dc6172549086a2",
      "sender": "luke-jr",
      "payload": "XD",
      "action": false,
      "timestamp": "2017-11-30T19:13:35+00:00"
    },
    {
      "id": "3c96cc402a4642cb8ec5128716319394",
      "sender": "wumpus",
      "payload": "using bitcoin keys for anything else than signing transactions continues to make me uneasy",
      "action": false,
      "timestamp": "2017-11-30T19:13:49+00:00"
    },
    {
      "id": "e1a6472bb9f046e1b7710d0f1f077569",
      "sender": "jtimon",
      "payload": "yeah, +1 to move to signing messages based on scripts",
      "action": false,
      "timestamp": "2017-11-30T19:13:50+00:00"
    },
    {
      "id": "8dd93b7d1edb4f01858ce964611abfe7",
      "sender": "luke-jr",
      "payload": "MAST-based signmessage would be a sensible replacement",
      "action": false,
      "timestamp": "2017-11-30T19:14:09+00:00"
    },
    {
      "id": "bc3c917c204d4f049d64e21f3ce79081",
      "sender": "Provoostenator",
      "payload": "jtimon: what would that look like?",
      "action": false,
      "timestamp": "2017-11-30T19:14:14+00:00"
    },
    {
      "id": "416b6efbe00a46bdbf54542db8e01308",
      "sender": "wumpus",
      "payload": "do hardware wallets even support it?",
      "action": false,
      "timestamp": "2017-11-30T19:14:44+00:00"
    },
    {
      "id": "d2e41bd4bb034728bacc7764c0f68dbc",
      "sender": "luke-jr",
      "payload": "Provoostenator: you could have a MAST if-branch that is always false for transactions, and then true for meta uses",
      "action": false,
      "timestamp": "2017-11-30T19:14:51+00:00"
    },
    {
      "id": "e10968c881cd4231b8108b5b42da8e98",
      "sender": "gmaxwell",
      "payload": "wumpus: If I were to it again, I'd make signmessage work by just creating a transaction which is inherently unminable. It would make it a lot more compatible, though somewhat larger.",
      "action": false,
      "timestamp": "2017-11-30T19:14:53+00:00"
    },
    {
      "id": "fcbf9a9dc7be43db8290c783c076258b",
      "sender": "CodeShark",
      "payload": "gmaxwell: a general solution for airdrops seems extremely difficult",
      "action": false,
      "timestamp": "2017-11-30T19:14:53+00:00"
    },
    {
      "id": "4ec93a6fb8854c8db940c4c7e3a1eeb8",
      "sender": "jtimon",
      "payload": "Provoostenator: like we sign txs but signing any message, in elements we sign blocks so perhaps you want to take a look (since there's generic functions to sign stuff, but they're not exposed and only used for blocks)",
      "action": false,
      "timestamp": "2017-11-30T19:15:29+00:00"
    },
    {
      "id": "58380b4dba1b4759a3b1f7320dc948ac",
      "sender": "gmaxwell",
      "payload": "In elements sipa wrote a patch for a script based signmessage; but we ran into ... challenges.. with how softforks would be handled.",
      "action": false,
      "timestamp": "2017-11-30T19:15:30+00:00"
    },
    {
      "id": "9ff6645dc62f4e6fbdeea27137fa1af0",
      "sender": "wumpus",
      "payload": "gmaxwell: that'd have been better; at least the keys would only be used to sign things in one format",
      "action": false,
      "timestamp": "2017-11-30T19:15:50+00:00"
    },
    {
      "id": "7421ee5e98814cad92c1f9af775e219d",
      "sender": "gmaxwell",
      "payload": "Segwit script versioning would fix those challenges.",
      "action": false,
      "timestamp": "2017-11-30T19:15:56+00:00"
    },
    {
      "id": "6d0acf3c443249dda4ed57440bea8169",
      "sender": "CodeShark",
      "payload": "yes, two chains might have very different redemption conditions for the same utxo",
      "action": false,
      "timestamp": "2017-11-30T19:15:57+00:00"
    },
    {
      "id": "38ce82253e7b4282b38191ba6632fe4b",
      "sender": "Provoostenator",
      "payload": "Hah, so you can do multisig messages? :-)",
      "action": false,
      "timestamp": "2017-11-30T19:16:02+00:00"
    },
    {
      "id": "150c9797f28948e09872050bd30c9c14",
      "sender": "sipa",
      "payload": "Provoostenator: yes",
      "action": false,
      "timestamp": "2017-11-30T19:16:07+00:00"
    },
    {
      "id": "971429f862f84753b32b13bd53ba41a9",
      "sender": "sipa",
      "payload": "Provoostenator: and timelocked signatures :p",
      "action": false,
      "timestamp": "2017-11-30T19:16:15+00:00"
    },
    {
      "id": "f7abbc5ec40d496283857cc3ec928d29",
      "sender": "Provoostenator",
      "payload": "Or even partial messages?",
      "action": false,
      "timestamp": "2017-11-30T19:16:16+00:00"
    },
    {
      "id": "47a0841c830b40c199439968d573aeba",
      "sender": "jtimon",
      "payload": "gmaxwell: I think they should be handled with flags",
      "action": false,
      "timestamp": "2017-11-30T19:16:20+00:00"
    },
    {
      "id": "9fb8136e97fb4100b3f0c91905a4686d",
      "sender": "sipa",
      "payload": "jtimon: yes, but the general property of softforks doesn't apply",
      "action": false,
      "timestamp": "2017-11-30T19:16:43+00:00"
    },
    {
      "id": "8a4193c6a46a4c5dbc719cf5670ccc31",
      "sender": "jtimon",
      "payload": "oh, right, script versioning solves it too",
      "action": false,
      "timestamp": "2017-11-30T19:16:45+00:00"
    },
    {
      "id": "6932938b5c9f4561b73121bc50c21445",
      "sender": "sipa",
      "payload": "you don't want a \"oh, i don't know this signature version! it's valid!!\"",
      "action": false,
      "timestamp": "2017-11-30T19:16:56+00:00"
    },
    {
      "id": "f485c73594384ed4b5407fce5a5d14a2",
      "sender": "gmaxwell",
      "payload": "jtimon: the 'pubkey' needs to commit to the rules. pre-segwit style softforks don't.",
      "action": false,
      "timestamp": "2017-11-30T19:17:01+00:00"
    },
    {
      "id": "d1d09d3eb6694e35acc50ec623205b51",
      "sender": "gmaxwell",
      "payload": "Well you can tristate:  Valid, invalid, unknown public key version.",
      "action": false,
      "timestamp": "2017-11-30T19:17:16+00:00"
    },
    {
      "id": "a80e302755b6446e9a4cc1dfdc5d00cd",
      "sender": "wumpus",
      "payload": "right",
      "action": false,
      "timestamp": "2017-11-30T19:17:48+00:00"
    },
    {
      "id": "b3c6e1d9cdd94c16914135f7343955d2",
      "sender": "luke-jr",
      "payload": "sipa: that's a risk for txs too, though",
      "action": false,
      "timestamp": "2017-11-30T19:17:50+00:00"
    },
    {
      "id": "4c0716ad042847f6b811b572d8ca5894",
      "sender": "jtimon",
      "payload": "sipa: sure, I mean, this is out of the blockchain, so you just need to know which flags to use when validating...what gmaxwell said, commit to the rules",
      "action": false,
      "timestamp": "2017-11-30T19:17:54+00:00"
    },
    {
      "id": "e7e199734d0d44e0aac286985a09cd85",
      "sender": "gmaxwell",
      "payload": "prior to having the explicit versions we couldn't do that, which is why we never even merged the patch in elements, much less brought it to bitcoin.",
      "action": false,
      "timestamp": "2017-11-30T19:18:06+00:00"
    },
    {
      "id": "68e987e46e8b4b9ab661cdfc3b855142",
      "sender": "luke-jr",
      "payload": "why sighash flags can't fail valid",
      "action": false,
      "timestamp": "2017-11-30T19:18:07+00:00"
    },
    {
      "id": "8825819ddb1a4462a04f72fd9050fe14",
      "sender": "gmaxwell",
      "payload": "luke-jr: attacker controls signature side flags.",
      "action": false,
      "timestamp": "2017-11-30T19:18:24+00:00"
    },
    {
      "id": "a1fcfc6c15f04810b539357ff68f333d",
      "sender": "luke-jr",
      "payload": "gmaxwell: right, that's my point",
      "action": false,
      "timestamp": "2017-11-30T19:18:32+00:00"
    },
    {
      "id": "a4c0bd30945e44c3aae0421f4724d80a",
      "sender": "gmaxwell",
      "payload": "same reason sighash flags can fail valid in bitcoin: If I pick an out of range sighash flag I could forge signatures for your pubkeys.",
      "action": false,
      "timestamp": "2017-11-30T19:19:15+00:00"
    },
    {
      "id": "1c0315fba7274060bae8912a0e7daf68",
      "sender": "sipa",
      "payload": "?",
      "action": false,
      "timestamp": "2017-11-30T19:19:38+00:00"
    },
    {
      "id": "51763b38e4044ad286489747566f2f76",
      "sender": "CodeShark",
      "payload": "I also don't follow",
      "action": false,
      "timestamp": "2017-11-30T19:20:24+00:00"
    },
    {
      "id": "2920688f06a246b898b3de6c668a7331",
      "sender": "luke-jr",
      "payload": "[19:16:56] <sipa> you don't want a \"oh, i don't know this signature version! it's valid\u00c3\u00a2\u00c2\u0080\u00c2\u00bc\" <-- this is a problem for transactions just as much as for signed messages",
      "action": false,
      "timestamp": "2017-11-30T19:20:49+00:00"
    },
    {
      "id": "fdd74949d27049b7a861deea215b2052",
      "sender": "wumpus",
      "payload": "are there any topics we really want to discuss in this meeting? I think we're drifting off a bit",
      "action": false,
      "timestamp": "2017-11-30T19:20:50+00:00"
    },
    {
      "id": "843aed7c02804bc6883ecba022d94cab",
      "sender": "gmaxwell",
      "payload": "You cannot trigger \"unknown behavior, I'll let it pass\" based on sighash flags like you can based on the content of public keys.",
      "action": false,
      "timestamp": "2017-11-30T19:21:00+00:00"
    },
    {
      "id": "cb29c130134a437c87f46a9ca998de4d",
      "sender": "cfields",
      "payload": "next (quick) topic suggestion: codesigning keys update",
      "action": false,
      "timestamp": "2017-11-30T19:21:06+00:00"
    },
    {
      "id": "a757227c181c44e89c4ff348d7509e41",
      "sender": "jnewbery",
      "payload": "Other PRs I'd love to see merged for v0.16 are #10583 #10579 #11415 . They're all removing wallet dependencies from server, but are API changes so have a one release deprecation lag time before the dependency can actually be removed.",
      "action": false,
      "timestamp": "2017-11-30T19:21:07+00:00"
    },
    {
      "id": "b169e8be93394e8ab09322403eb9d494",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/10583 | [RPC] Split part of validateaddress into getaddressinfo by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #10583 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:21:10+00:00"
    },
    {
      "id": "d6dcb7fa0b58485398243d8d1afd96d9",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/10579 | [RPC] Split signrawtransaction into wallet and non-wallet RPC command by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #10579 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:21:14+00:00"
    },
    {
      "id": "1369bc6ae7fd4cc3812ce9034d715ed2",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/11415 | [RPC] Disallow using addresses in createmultisig by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #11415 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:21:17+00:00"
    },
    {
      "id": "6b25b5ae9bf94b09b67f9eb72ad06829",
      "sender": "jtimon",
      "payload": "yeah, how about what \"shortly after\" means for 0.16 ?",
      "action": false,
      "timestamp": "2017-11-30T19:21:18+00:00"
    },
    {
      "id": "96f133ce56d84c0c9d4f1706c591dfdb",
      "sender": "jonasschnelli",
      "payload": "Two topic proposals: 1) review \"nits\" reduction, 2) bitcoin core code signing assoc.",
      "action": false,
      "timestamp": "2017-11-30T19:21:24+00:00"
    },
    {
      "id": "ac7d2b1c47d143189289426d150090eb",
      "sender": "jonasschnelli",
      "payload": "2) goes in hand with cfields topicp.",
      "action": false,
      "timestamp": "2017-11-30T19:21:39+00:00"
    },
    {
      "id": "38873041f85c456082993d63177d975c",
      "sender": "wumpus",
      "payload": "jnewbery: thanks, I think adding all of them to high priority is a bit much, but we could add one",
      "action": false,
      "timestamp": "2017-11-30T19:22:07+00:00"
    },
    {
      "id": "b29d1a93772f477fb0976a9c8790153d",
      "sender": "wumpus",
      "payload": "#topic codesigning",
      "action": false,
      "timestamp": "2017-11-30T19:22:09+00:00"
    },
    {
      "id": "7bd008618a4f4d8a9a28bdd0310014cb",
      "sender": "cfields",
      "payload": "I just wanted to point out that after researching for a bit, there's no painless way to renew our osx cert (without involving the btcf), so I think it's prudent that we explore other options.",
      "action": false,
      "timestamp": "2017-11-30T19:22:17+00:00"
    },
    {
      "id": "7de1151d75b5437598b3085c055c7662",
      "sender": "cfields",
      "payload": "passes the mic to jonasschnelli",
      "action": true,
      "timestamp": "2017-11-30T19:22:22+00:00"
    },
    {
      "id": "da51433b08cd4a29adeff3fbe0ecaad4",
      "sender": "jonasschnelli",
      "payload": "https://github.com/bitcoincore-codesigning-association",
      "action": false,
      "timestamp": "2017-11-30T19:22:28+00:00"
    },
    {
      "id": "93cac8123f7b4c4f924ca3dbf39ca13c",
      "sender": "achow101",
      "payload": "wumpus: it seems like some of those are ready to be merged. they have utACKs and no comments left",
      "action": false,
      "timestamp": "2017-11-30T19:22:44+00:00"
    },
    {
      "id": "33572be694de46ec921276e2bb6cc135",
      "sender": "jonasschnelli",
      "payload": "https://bitcoincorecodesigning.org",
      "action": false,
      "timestamp": "2017-11-30T19:22:45+00:00"
    },
    {
      "id": "7c43cf01226b468c8706ab6a21481eb3",
      "sender": "jonasschnelli",
      "payload": "The association stands and apple has accepted the entity",
      "action": false,
      "timestamp": "2017-11-30T19:22:45+00:00"
    },
    {
      "id": "f9378844ad9244caa779c809d9711fa5",
      "sender": "jonasschnelli",
      "payload": "Code sining certificates are ready (need to setup the key)",
      "action": false,
      "timestamp": "2017-11-30T19:22:57+00:00"
    },
    {
      "id": "c394d838cdd147f6b4006649e49be17a",
      "sender": "cfields",
      "payload": "jonasschnelli: oh that's fantastic! nice work :)",
      "action": false,
      "timestamp": "2017-11-30T19:23:12+00:00"
    },
    {
      "id": "72719fdb4deb4040a417dd7c64d5b74f",
      "sender": "wumpus",
      "payload": "achow101: if they're ready for merge even better, please notify me of those things outside the meeting",
      "action": false,
      "timestamp": "2017-11-30T19:23:15+00:00"
    },
    {
      "id": "86a9e224483f419aae5aaac50244f563",
      "sender": "jnewbery",
      "payload": "wumpus: thanks. I think they're all pretty much ready for merge. achow101 has been doing a great job maintaining and rebasing them - perhaps just a bit more ACKing and they can go in",
      "action": false,
      "timestamp": "2017-11-30T19:23:16+00:00"
    },
    {
      "id": "490101d2e6fc4e3ab1063e4650b902ff",
      "sender": "jonasschnelli",
      "payload": "We need a windows code signing cert... have never done that but I'm ready to order if someone gives instructions where and how",
      "action": false,
      "timestamp": "2017-11-30T19:23:47+00:00"
    },
    {
      "id": "9bcb7319f16349ea8650bb7857ea9f47",
      "sender": "wumpus",
      "payload": "PSA: if something is ready for merge just let me know here instead of waiting for me to stumble on it accidentally :)",
      "action": false,
      "timestamp": "2017-11-30T19:24:00+00:00"
    },
    {
      "id": "97fc8fae794b4787beda5dc6c1d3bc67",
      "sender": "BlueMatt",
      "payload": "jnewbery: afaict only maybe the last is ready? the second only has one ack?",
      "action": false,
      "timestamp": "2017-11-30T19:24:17+00:00"
    },
    {
      "id": "abf8ebad95e44d7ca22244b9553b3274",
      "sender": "achow101",
      "payload": "wumpus: ok!",
      "action": false,
      "timestamp": "2017-11-30T19:24:22+00:00"
    },
    {
      "id": "94eff5ec36834a0cbcc091eee48e602d",
      "sender": "BlueMatt",
      "payload": "jonasschnelli: that was fast!",
      "action": false,
      "timestamp": "2017-11-30T19:24:25+00:00"
    },
    {
      "id": "9855f9ae3db141c7b14d1c6ca493a8fa",
      "sender": "wumpus",
      "payload": "jonasschnelli: oh great!",
      "action": false,
      "timestamp": "2017-11-30T19:24:36+00:00"
    },
    {
      "id": "9ca164927ca144d99630e463c49afcc1",
      "sender": "achow101",
      "payload": "jonasschnelli: https://docs.microsoft.com/en-us/windows-hardware/drivers/dashboard/get-a-code-signing-certificate",
      "action": false,
      "timestamp": "2017-11-30T19:24:37+00:00"
    },
    {
      "id": "fddc90871a1c4a049a0f492d1d9a5efc",
      "sender": "cfields",
      "payload": "achow101: note that a driver cert is different. More requirements.",
      "action": false,
      "timestamp": "2017-11-30T19:25:15+00:00"
    },
    {
      "id": "9722a02b1ce94525965a28d131236551",
      "sender": "wumpus",
      "payload": "getting a driver cert is much more difficult",
      "action": false,
      "timestamp": "2017-11-30T19:25:28+00:00"
    },
    {
      "id": "da971203e3f442e492fed6af8991098d",
      "sender": "achow101",
      "payload": "oh, didn't see that that was for drivers",
      "action": false,
      "timestamp": "2017-11-30T19:25:42+00:00"
    },
    {
      "id": "6eee4b8f1e764ec69ad6d5996015635c",
      "sender": "wumpus",
      "payload": "I guess it should be, with the privilege level it operates at",
      "action": false,
      "timestamp": "2017-11-30T19:25:47+00:00"
    },
    {
      "id": "721ef875de5844d18c96d1d6824390de",
      "sender": "jonasschnelli",
      "payload": "The question is if we want to try that distributed RSA key",
      "action": false,
      "timestamp": "2017-11-30T19:26:01+00:00"
    },
    {
      "id": "a1a3fe88672d4ae9968b22b3f988d07f",
      "sender": "wumpus",
      "payload": "we do",
      "action": false,
      "timestamp": "2017-11-30T19:26:09+00:00"
    },
    {
      "id": "f171e0e99eb7467d84f44a5df95f8c71",
      "sender": "BlueMatt",
      "payload": "gmaxwell: mic?",
      "action": false,
      "timestamp": "2017-11-30T19:26:26+00:00"
    },
    {
      "id": "7c0d69a379114931838f200619f0948e",
      "sender": "BlueMatt",
      "payload": "or who was it who said they were gonna look into hooking it into a reasonable way to use it?",
      "action": false,
      "timestamp": "2017-11-30T19:26:48+00:00"
    },
    {
      "id": "e5e96a5c53d94eb8b268f660bbde1bda",
      "sender": "cfields",
      "payload": "I'll work on the CSR handling as promised. I assume we'll just have to shove the result back into the expected format.",
      "action": false,
      "timestamp": "2017-11-30T19:26:54+00:00"
    },
    {
      "id": "181df91301da4268b21c63de8565c076",
      "sender": "gmaxwell",
      "payload": "I didn't think we were going to bother to do it for the first one. but this is going faster than expected! :)",
      "action": false,
      "timestamp": "2017-11-30T19:27:21+00:00"
    },
    {
      "id": "c2f87ab0bc804d3da301697a20343b32",
      "sender": "jonasschnelli",
      "payload": "cfields: Yeah. Apple needs that -----BEGIN CERTIFICATE REQUEST-----",
      "action": false,
      "timestamp": "2017-11-30T19:27:22+00:00"
    },
    {
      "id": "3663cb092c9d4a8088aa2af72dcb2e94",
      "sender": "achow101",
      "payload": "BlueMatt: that was gmaxwell. I also wanted to take a look at it. it looked painful",
      "action": false,
      "timestamp": "2017-11-30T19:27:23+00:00"
    },
    {
      "id": "e6bac73ac7e44acca240dc35132fc398",
      "sender": "gmaxwell",
      "payload": "it's not so painful but we need a process for converting raw RSA numbers into csrs and what not, which cfields was going to look into.",
      "action": false,
      "timestamp": "2017-11-30T19:27:58+00:00"
    },
    {
      "id": "24cfc3e5779d432c9b4623d6e8cd4d3f",
      "sender": "gmaxwell",
      "payload": "The easiest thing to do may be to just do trusted setup: generate the key normally on an offline machine, and we can split it after the the fact.",
      "action": false,
      "timestamp": "2017-11-30T19:28:30+00:00"
    },
    {
      "id": "4097d23c47904abfaa41b80b017e170c",
      "sender": "BlueMatt",
      "payload": "ugh",
      "action": false,
      "timestamp": "2017-11-30T19:28:48+00:00"
    },
    {
      "id": "a2b60f79e1774b32ad7cfde2cbe7b7fc",
      "sender": "gmaxwell",
      "payload": "the MPC signing is radically simple and faster than MPC key generation.",
      "action": false,
      "timestamp": "2017-11-30T19:28:50+00:00"
    },
    {
      "id": "3317e96c06c24dc19c5f1ae849649044",
      "sender": "BlueMatt",
      "payload": "i thought the mpc generation was implemented?",
      "action": false,
      "timestamp": "2017-11-30T19:29:01+00:00"
    },
    {
      "id": "118c958224b5445780700245294edb2f",
      "sender": "gmaxwell",
      "payload": "Though the stuff I linked to does both. (though it's setup for only three parties atm)",
      "action": false,
      "timestamp": "2017-11-30T19:29:08+00:00"
    },
    {
      "id": "049e0610612846ad8e28a3569071fd4f",
      "sender": "BlueMatt",
      "payload": "i mean thats probably fine?",
      "action": false,
      "timestamp": "2017-11-30T19:29:17+00:00"
    },
    {
      "id": "e5ad07209b83482a84f00a9927e002e3",
      "sender": "BlueMatt",
      "payload": "3/3, I assume?",
      "action": false,
      "timestamp": "2017-11-30T19:29:22+00:00"
    },
    {
      "id": "490c143ee7c54d129c4a13e5daaba343",
      "sender": "achow101",
      "payload": "gmaxwell: can it be done faster than what is already implemented?",
      "action": false,
      "timestamp": "2017-11-30T19:29:22+00:00"
    },
    {
      "id": "4c7488919a8047db924e48f915854b9f",
      "sender": "gmaxwell",
      "payload": "achow101: of course, but do we care if it takes hours?",
      "action": false,
      "timestamp": "2017-11-30T19:29:35+00:00"
    },
    {
      "id": "aaed3359d81244dda8a122a70266bab7",
      "sender": "wumpus",
      "payload": "if it's a one-time thing we don't",
      "action": false,
      "timestamp": "2017-11-30T19:30:12+00:00"
    },
    {
      "id": "dd497c30edf244e58647bb63634afe5d",
      "sender": "cfields",
      "payload": "jonasschnelli: let's talk after the meeting. It'd be great if we could get a dummy to test with.",
      "action": false,
      "timestamp": "2017-11-30T19:30:15+00:00"
    },
    {
      "id": "64153dab51f7417ba39929b41b74d7b0",
      "sender": "gmaxwell",
      "payload": "BlueMatt: yes 3/3. Generation you'd always do as n/n then potentially reshare to a different threshold.",
      "action": false,
      "timestamp": "2017-11-30T19:30:19+00:00"
    },
    {
      "id": "8c8f85e23571466cbb583e377f4dd7c4",
      "sender": "jonasschnelli",
      "payload": "cfields: sure",
      "action": false,
      "timestamp": "2017-11-30T19:30:25+00:00"
    },
    {
      "id": "3b807b23fbe34d29858f3c7951c0fdb5",
      "sender": "BlueMatt",
      "payload": "its not like this is holding a zksnark trusted-setup for all btc",
      "action": false,
      "timestamp": "2017-11-30T19:30:27+00:00"
    },
    {
      "id": "08fee676c65d415ab6bd98223bf89391",
      "sender": "gmaxwell",
      "payload": "How big are the keys they use for these things? 2kbit or 4kbit?",
      "action": false,
      "timestamp": "2017-11-30T19:30:37+00:00"
    },
    {
      "id": "9f1bd6778e8d48638afff7f2c0b1d4ed",
      "sender": "Provoostenator",
      "payload": "BlueMatt: would be nice to get another blog from P",
      "action": false,
      "timestamp": "2017-11-30T19:31:02+00:00"
    },
    {
      "id": "293ad3276f1a4a4abf4546ef5ef07f9d",
      "sender": "Provoostenator",
      "payload": "l",
      "action": false,
      "timestamp": "2017-11-30T19:31:02+00:00"
    },
    {
      "id": "65529c3cd872452ab9adb98cb32fd4f1",
      "sender": "sipa",
      "payload": "?",
      "action": false,
      "timestamp": "2017-11-30T19:31:11+00:00"
    },
    {
      "id": "d6be45a5be694e6d9aaad18a88320954",
      "sender": "BlueMatt",
      "payload": "?",
      "action": false,
      "timestamp": "2017-11-30T19:31:13+00:00"
    },
    {
      "id": "222bdde087014dcda5afef5be0b68e51",
      "sender": "Provoostenator",
      "payload": "Peter todd riding down Canada to generate his signing key.",
      "action": false,
      "timestamp": "2017-11-30T19:31:19+00:00"
    },
    {
      "id": "4ed04458186f4be3867e2ceee176e284",
      "sender": "jonasschnelli",
      "payload": "gmaxwell: 2048 is what my apple RSA keys are",
      "action": false,
      "timestamp": "2017-11-30T19:31:29+00:00"
    },
    {
      "id": "996d32a1f6fa43c2a84abe26dfe13802",
      "sender": "Provoostenator",
      "payload": "And then bragging that's more secure than Zcash",
      "action": false,
      "timestamp": "2017-11-30T19:31:33+00:00"
    },
    {
      "id": "17a8ba4e05a843569048174770fc56a6",
      "sender": "wumpus",
      "payload": "indeed, it's just code signing, for one OS, if something is signed that is not validated by the gitian process that's discovered soon enough anyway",
      "action": false,
      "timestamp": "2017-11-30T19:31:36+00:00"
    },
    {
      "id": "f63a47e49825439eae7c2f15edb91d0d",
      "sender": "gmaxwell",
      "payload": "Provoostenator: it's totally unrelated.",
      "action": false,
      "timestamp": "2017-11-30T19:31:38+00:00"
    },
    {
      "id": "636e8a7ecba94d89928545ff7da6719b",
      "sender": "jonasschnelli",
      "payload": "Code signing won't give a lot of additional security...",
      "action": false,
      "timestamp": "2017-11-30T19:31:58+00:00"
    },
    {
      "id": "7e795876afc04024a4b2622c90dbb9e5",
      "sender": "jonasschnelli",
      "payload": "It's just a UX thing IMO",
      "action": false,
      "timestamp": "2017-11-30T19:32:02+00:00"
    },
    {
      "id": "dd036846266f4c0c8dc7a9be96fd8d8c",
      "sender": "gmaxwell",
      "payload": "zcash has a backdoor for the whole system, this is just some code signing crud.",
      "action": false,
      "timestamp": "2017-11-30T19:32:07+00:00"
    },
    {
      "id": "739698bb549f4bf39bf1a1320ef4ea0f",
      "sender": "wumpus",
      "payload": "right",
      "action": false,
      "timestamp": "2017-11-30T19:32:25+00:00"
    },
    {
      "id": "1d4977b6756c4942b80561d3c349449a",
      "sender": "Provoostenator",
      "payload": "I know.",
      "action": false,
      "timestamp": "2017-11-30T19:32:29+00:00"
    },
    {
      "id": "9bbc1fc6d8164627942bb245e3c1baca",
      "sender": "gmaxwell",
      "payload": "the users shouldn't care AT ALL about the MPC we use for it, the purpose of the MPC is to protect the developers personally from being implicated in the unfortunate case their own computers get hacked.",
      "action": false,
      "timestamp": "2017-11-30T19:32:43+00:00"
    },
    {
      "id": "09b2f6eb03664884ba6c43e1fff50ac6",
      "sender": "sipa",
      "payload": "does the MPC generation have a trusted setup?",
      "action": false,
      "timestamp": "2017-11-30T19:32:53+00:00"
    },
    {
      "id": "1cf1dba4f7af4303b56e6aed5c35c139",
      "sender": "gmaxwell",
      "payload": "sipa: no.",
      "action": false,
      "timestamp": "2017-11-30T19:32:59+00:00"
    },
    {
      "id": "3d9850d7046f439ba5588c013e26b0b5",
      "sender": "sipa",
      "payload": "i assume not - if you're fine with trusted setup, just generate a single key and split it layer",
      "action": false,
      "timestamp": "2017-11-30T19:33:07+00:00"
    },
    {
      "id": "454c214b48c74c1f8aff935c087b06c9",
      "sender": "sipa",
      "payload": "ok, so it is entirely unrelated",
      "action": false,
      "timestamp": "2017-11-30T19:33:13+00:00"
    },
    {
      "id": "2529a06f3f764c74863383ca86f3c4b1",
      "sender": "jtimon",
      "payload": "is curious about the \"review nits reduction\" topic...",
      "action": true,
      "timestamp": "2017-11-30T19:33:15+00:00"
    },
    {
      "id": "ef241a31d59f41039d49e6b58247db2f",
      "sender": "wumpus",
      "payload": "yes it's just to prevent the person doing the code signing from becoming a specific target",
      "action": false,
      "timestamp": "2017-11-30T19:33:25+00:00"
    },
    {
      "id": "08075d3423994379bfe7f976aabd5e70",
      "sender": "jonasschnelli",
      "payload": "review nit reduction topic?",
      "action": false,
      "timestamp": "2017-11-30T19:34:12+00:00"
    },
    {
      "id": "d6fbd848ec5d4644a1c0cf5b039c530d",
      "sender": "gmaxwell",
      "payload": "sipa: it just uses paillier, and lots of roundtrips. it's pretty simple.",
      "action": false,
      "timestamp": "2017-11-30T19:34:25+00:00"
    },
    {
      "id": "1d76cd765b5345c68e1bbb731fac6fb3",
      "sender": "wumpus",
      "payload": "#topic review nit reduction",
      "action": false,
      "timestamp": "2017-11-30T19:34:48+00:00"
    },
    {
      "id": "c6c4da4039674d6cabdc2fa6325bf34e",
      "sender": "jonasschnelli",
      "payload": "I just feel that we spend a lot of time in finding code-style nits, fixing code-style nits and some of the PRs are almost a single wall of nits... It doesn't seem to be efficient... I think..",
      "action": false,
      "timestamp": "2017-11-30T19:35:10+00:00"
    },
    {
      "id": "f5ae70cf84194a2bb9c99196fb8f9543",
      "sender": "jonasschnelli",
      "payload": "We should enforce the rules... ( I know I already brought that up )",
      "action": false,
      "timestamp": "2017-11-30T19:35:30+00:00"
    },
    {
      "id": "0a732af30e2846179a142617a8c31551",
      "sender": "jonasschnelli",
      "payload": "The libcurl project does that... it won't compile (make) if the code style doesn't matches",
      "action": false,
      "timestamp": "2017-11-30T19:36:03+00:00"
    },
    {
      "id": "aa9455ae9fe84ca98bf77ec8524742a3",
      "sender": "jonasschnelli",
      "payload": "It would reduce the review time a lot...",
      "action": false,
      "timestamp": "2017-11-30T19:36:10+00:00"
    },
    {
      "id": "a84d6ffa772a4a56864b9b05d8f5edb0",
      "sender": "jonasschnelli",
      "payload": "And we apperently fixing the nits anyways at some point in time",
      "action": false,
      "timestamp": "2017-11-30T19:36:20+00:00"
    },
    {
      "id": "5ac2143e50834e1890bc9fb7cf213e8f",
      "sender": "jtimon",
      "payload": "you mean getting travis to complain about style nits?",
      "action": false,
      "timestamp": "2017-11-30T19:36:52+00:00"
    },
    {
      "id": "75beada05e9849af992dd6b344d5c146",
      "sender": "Provoostenator",
      "payload": "Currently we're only running whitespace linter?",
      "action": false,
      "timestamp": "2017-11-30T19:36:56+00:00"
    },
    {
      "id": "1b324c899e0d4f1d943918f09b233a8d",
      "sender": "jonasschnelli",
      "payload": "jtimon: travis already does this... (whitespace)",
      "action": false,
      "timestamp": "2017-11-30T19:37:10+00:00"
    },
    {
      "id": "82078398c2224db1ac4d226c70be09c2",
      "sender": "BlueMatt",
      "payload": "if there were an easy way to apply it to new code only, I'd say thats probably fine",
      "action": false,
      "timestamp": "2017-11-30T19:37:18+00:00"
    },
    {
      "id": "c3c2c6cd5b7044fe862f887d80224b9f",
      "sender": "jonasschnelli",
      "payload": "I just want to get the code styling nits away from github comments",
      "action": false,
      "timestamp": "2017-11-30T19:37:27+00:00"
    },
    {
      "id": "1d01b1aef7ff41029cb7318e94bc917d",
      "sender": "jtimon",
      "payload": "nods",
      "action": true,
      "timestamp": "2017-11-30T19:37:28+00:00"
    },
    {
      "id": "2720849292dc4065a1046ad7400eea4a",
      "sender": "BlueMatt",
      "payload": "but it was my impression we were not able to find a way to do so",
      "action": false,
      "timestamp": "2017-11-30T19:37:33+00:00"
    },
    {
      "id": "97778ac888134d7c9a536efa2f444af7",
      "sender": "jonasschnelli",
      "payload": "BlueMatt: only new code. Yes.",
      "action": false,
      "timestamp": "2017-11-30T19:37:42+00:00"
    },
    {
      "id": "c8fce1b46ca14c10828b25f18cd6437e",
      "sender": "jtimon",
      "payload": "BlueMatt: that was my understanding too",
      "action": false,
      "timestamp": "2017-11-30T19:37:47+00:00"
    },
    {
      "id": "4c741ac0238245f7b90d72e172f71234",
      "sender": "wumpus",
      "payload": "so have e.g. clang-format check the diff of pull requests?",
      "action": false,
      "timestamp": "2017-11-30T19:38:03+00:00"
    },
    {
      "id": "1eeef05ec78740958ea51e20065b38ee",
      "sender": "sipa",
      "payload": "jonasschnelli: (brainstorming) or we could have a bot that marks a PR ready for review only after all nits are fixed",
      "action": false,
      "timestamp": "2017-11-30T19:38:12+00:00"
    },
    {
      "id": "511fd5087c334b10adb8f4727f99b25d",
      "sender": "jonasschnelli",
      "payload": "wumpus: Yes. Can we enforce that somehow?",
      "action": false,
      "timestamp": "2017-11-30T19:38:15+00:00"
    },
    {
      "id": "a07e271e3e254d3e9a7d19a4d1fdf068",
      "sender": "jonasschnelli",
      "payload": "sipa: Would also be something.. yes.",
      "action": false,
      "timestamp": "2017-11-30T19:38:28+00:00"
    },
    {
      "id": "345f65e5cb5b4e709b433d10159683f9",
      "sender": "sipa",
      "payload": "so that people know not to look at something while there is still automated review going on",
      "action": false,
      "timestamp": "2017-11-30T19:38:31+00:00"
    },
    {
      "id": "a6982187da8a4f9e9946baafa468c7f0",
      "sender": "wumpus",
      "payload": "jonasschnelli: I don't know...",
      "action": false,
      "timestamp": "2017-11-30T19:38:32+00:00"
    },
    {
      "id": "de9e285311f742d6b346a86bb79758e6",
      "sender": "Provoostenator",
      "payload": "It would also help to offer some hints for developers how to run linters in their editor. I'll add an entry for OSX Atom once I figure it out myself.",
      "action": false,
      "timestamp": "2017-11-30T19:38:33+00:00"
    },
    {
      "id": "d79e3f22b51146d29aac59cee7222afa",
      "sender": "jtimon",
      "payload": "didn't someone wrote something like that at some point?",
      "action": false,
      "timestamp": "2017-11-30T19:38:35+00:00"
    },
    {
      "id": "d940ab4b0bfa4894b48e630df353e3d2",
      "sender": "wumpus",
      "payload": "sipa: as long as it doesn't generate noise (posts) that's ok with me",
      "action": false,
      "timestamp": "2017-11-30T19:38:41+00:00"
    },
    {
      "id": "21439c631eb6452296e73dc3acbbcff9",
      "sender": "wumpus",
      "payload": "sipa: so if it works like travis, just adds another cross to the status",
      "action": false,
      "timestamp": "2017-11-30T19:38:55+00:00"
    },
    {
      "id": "203ba04a53aa4c17b146a78b36036674",
      "sender": "jonasschnelli",
      "payload": "Would something speak against enforcing the clang-format-diff check during make?",
      "action": false,
      "timestamp": "2017-11-30T19:38:59+00:00"
    },
    {
      "id": "306f3d466b32471ab2d5cd5c12d858b4",
      "sender": "wumpus",
      "payload": "w/ a link to the list of problems",
      "action": false,
      "timestamp": "2017-11-30T19:39:02+00:00"
    },
    {
      "id": "b95873c1cae04dcfb8b8c5955f64ffc8",
      "sender": "sipa",
      "payload": "wumpus: right,t hat would be nice",
      "action": false,
      "timestamp": "2017-11-30T19:39:02+00:00"
    },
    {
      "id": "d50bc0510c014144869067837bfbde81",
      "sender": "meshcollider",
      "payload": "BlueMatt: adding new linters to Travis would only affect PRs now since #11699",
      "action": false,
      "timestamp": "2017-11-30T19:39:08+00:00"
    },
    {
      "id": "220a0cfb6afe4accb14c5ba3a161d20b",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/11699 | [travis-ci] Only run linters on Pull Requests by jnewbery \u00c3\u0082\u00c2\u00b7 Pull Request #11699 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:39:10+00:00"
    },
    {
      "id": "20f2403e913b49719fa048ba97a585a8",
      "sender": "wumpus",
      "payload": "but I don't want any bots that generate notifications in my mail",
      "action": false,
      "timestamp": "2017-11-30T19:39:14+00:00"
    },
    {
      "id": "32c64f34db424af894e9db8c3855cfe9",
      "sender": "sipa",
      "payload": "right",
      "action": false,
      "timestamp": "2017-11-30T19:39:32+00:00"
    },
    {
      "id": "b746475ed7e64ad2bc84500f987d8764",
      "sender": "jonasschnelli",
      "payload": "wumpus: Yes, That would disturb even more",
      "action": false,
      "timestamp": "2017-11-30T19:39:33+00:00"
    },
    {
      "id": "526d2caa699c4036973e1be4eda4eb1a",
      "sender": "jonasschnelli",
      "payload": "Just an indication if its ready to review...",
      "action": false,
      "timestamp": "2017-11-30T19:39:46+00:00"
    },
    {
      "id": "7ba981af2cd74b419fac96d2e0577b8e",
      "sender": "jonasschnelli",
      "payload": "And so we can have less of those \"brackets, spaces missing blabla\"",
      "action": false,
      "timestamp": "2017-11-30T19:39:59+00:00"
    },
    {
      "id": "4ba1f4f6e19947c7b096d47841c2beeb",
      "sender": "BlueMatt",
      "payload": "was always in favor, did we find out if we could get a reasonably stable output out of clang-format?",
      "action": true,
      "timestamp": "2017-11-30T19:40:32+00:00"
    },
    {
      "id": "51412d3919454c9da53a2ba329054e7c",
      "sender": "BlueMatt",
      "payload": "or was it always very version-dependant?",
      "action": false,
      "timestamp": "2017-11-30T19:40:38+00:00"
    },
    {
      "id": "486594c31126414dbd825f20faa32a35",
      "sender": "jonasschnelli",
      "payload": "It just feels some reviews are more or less a \"clang-format diff output\"",
      "action": false,
      "timestamp": "2017-11-30T19:40:47+00:00"
    },
    {
      "id": "6e553b9e215c45c096b19dd1e6ef2a55",
      "sender": "wumpus",
      "payload": "I like how golang handles that, they just have one true style",
      "action": false,
      "timestamp": "2017-11-30T19:40:52+00:00"
    },
    {
      "id": "30929a6b1ca7475582a05ad53230fdf5",
      "sender": "wumpus",
      "payload": "and their linter can be safely used in e.g. pull request checks, w/ no ambiguity",
      "action": false,
      "timestamp": "2017-11-30T19:41:12+00:00"
    },
    {
      "id": "a7f4c3ca57664e2296a56a53c64bda8c",
      "sender": "jtimon",
      "payload": "jonasschnelli: wouldn't we need to comply with clang format in the whole project before doing the clang check in make? has anyone seen haw many changes trying to apply clang-format to the whole project produces right now?",
      "action": false,
      "timestamp": "2017-11-30T19:41:14+00:00"
    },
    {
      "id": "8447e57aaadf495b8435694aeb9977f8",
      "sender": "morcos",
      "payload": "do we have good developer documentation on how to do the right thing locally (not what the rules are, but how to check them?)",
      "action": false,
      "timestamp": "2017-11-30T19:41:29+00:00"
    },
    {
      "id": "ae61cfa2273f49bb9e0a5203cc286c2e",
      "sender": "sipa",
      "payload": "you can apply clang-format on just the diffs",
      "action": false,
      "timestamp": "2017-11-30T19:41:32+00:00"
    },
    {
      "id": "37e91ae044b04a2ebc06f9e144af6f01",
      "sender": "meshcollider",
      "payload": "jtimon: not if it's only run on the diff",
      "action": false,
      "timestamp": "2017-11-30T19:41:37+00:00"
    },
    {
      "id": "8f5df98a6ebf4b0cb5f02b34c57357f6",
      "sender": "jonasschnelli",
      "payload": "yes",
      "action": false,
      "timestamp": "2017-11-30T19:41:44+00:00"
    },
    {
      "id": "e0b8ad9b41134126b341748fb2684f9f",
      "sender": "sipa",
      "payload": "but clang-format only checks whitespace/newlines",
      "action": false,
      "timestamp": "2017-11-30T19:41:47+00:00"
    },
    {
      "id": "07e22e3062474e268ca690c290988bb3",
      "sender": "wumpus",
      "payload": "morcos: good point!",
      "action": false,
      "timestamp": "2017-11-30T19:41:47+00:00"
    },
    {
      "id": "971bc859111541329446943eb317011e",
      "sender": "sipa",
      "payload": "it doesn't check variable names etc",
      "action": false,
      "timestamp": "2017-11-30T19:41:55+00:00"
    },
    {
      "id": "7aa150f8b3cf4bc9b5415d4c9077b056",
      "sender": "jonasschnelli",
      "payload": "morcos: we should focus on that",
      "action": false,
      "timestamp": "2017-11-30T19:41:56+00:00"
    },
    {
      "id": "0fd46f8b859346b783ceb07cccb75072",
      "sender": "gmaxwell",
      "payload": "A small amount of code style nits on github are probably good for teamwork.",
      "action": false,
      "timestamp": "2017-11-30T19:41:57+00:00"
    },
    {
      "id": "378072b026674e548dc23e10017b9a41",
      "sender": "wumpus",
      "payload": "sipa: I think that's fine, those are the ones we want out of the way",
      "action": false,
      "timestamp": "2017-11-30T19:42:03+00:00"
    },
    {
      "id": "0c490b8b85c74b4ab1934e312a87486d",
      "sender": "jonasschnelli",
      "payload": "sipa: not sure if that is possible",
      "action": false,
      "timestamp": "2017-11-30T19:42:04+00:00"
    },
    {
      "id": "0bf56181acdb4c50929d70820b2d59db",
      "sender": "jtimon",
      "payload": "meshcollider: right, only for pr diffs, yeah, I thought he meant every time you build locally",
      "action": false,
      "timestamp": "2017-11-30T19:42:06+00:00"
    },
    {
      "id": "7905dea1b5d5462d81ca98e1d96d4eba",
      "sender": "wumpus",
      "payload": "sipa: variable names are more of a human thing anyhow",
      "action": false,
      "timestamp": "2017-11-30T19:42:11+00:00"
    },
    {
      "id": "ab8d557887f841e594c9b158634fb7f8",
      "sender": "wumpus",
      "payload": "sipa: so it's fine if humans comment on them",
      "action": false,
      "timestamp": "2017-11-30T19:42:17+00:00"
    },
    {
      "id": "7495a76ed3154c1c9d8472e3e38b3053",
      "sender": "morcos",
      "payload": "i'd be happy to do more locally, i'm sure my PR's are disasters, but it would be nice if a good workflow was spelled out",
      "action": false,
      "timestamp": "2017-11-30T19:42:29+00:00"
    },
    {
      "id": "8f20c1facf2a4ec690cb164cc698d872",
      "sender": "jonasschnelli",
      "payload": "morcos: Indeed",
      "action": false,
      "timestamp": "2017-11-30T19:42:44+00:00"
    },
    {
      "id": "e87a1871d4504727912834d96285332a",
      "sender": "meshcollider",
      "payload": "Scripted diffs will probably regularly require style change commits too to keep Travis happy",
      "action": false,
      "timestamp": "2017-11-30T19:42:48+00:00"
    },
    {
      "id": "d7fff27246ae40e88d3b213fe603afa3",
      "sender": "jtimon",
      "payload": "gmaxwell: there's always style nits that go beyond clang-format",
      "action": false,
      "timestamp": "2017-11-30T19:42:57+00:00"
    },
    {
      "id": "c1c2326690264c47beb96918e7ca7cd9",
      "sender": "wumpus",
      "payload": "I mean it could check basic things like is_this_snake_case for variable names in theory, but not whether it's a good name in the first place...",
      "action": false,
      "timestamp": "2017-11-30T19:43:05+00:00"
    },
    {
      "id": "f4aab4d66f2046128a5165cf0bf6eea4",
      "sender": "sipa",
      "payload": "sur",
      "action": false,
      "timestamp": "2017-11-30T19:43:13+00:00"
    },
    {
      "id": "04e10776555b4cf8999b13b712876a6f",
      "sender": "jonasschnelli",
      "payload": "yes",
      "action": false,
      "timestamp": "2017-11-30T19:43:16+00:00"
    },
    {
      "id": "673315a3b8de4398ae3a55635026ed70",
      "sender": "jonasschnelli",
      "payload": "It's more about the naming convenction (m_, snake_case, CONSTANTS, etc,)",
      "action": false,
      "timestamp": "2017-11-30T19:43:35+00:00"
    },
    {
      "id": "48f8467f70fd4642b5fc3774693937d2",
      "sender": "wumpus",
      "payload": "and the latter is much more important for maintainablility :-)",
      "action": false,
      "timestamp": "2017-11-30T19:43:36+00:00"
    },
    {
      "id": "cc74d226761c4dd2b147fec4840e45f9",
      "sender": "kanzure",
      "payload": "do we have a clang-format file",
      "action": false,
      "timestamp": "2017-11-30T19:43:48+00:00"
    },
    {
      "id": "db0bd47acee94e7bbf0d8990fdb8c04c",
      "sender": "MarcoFalke",
      "payload": "meshcollider: I'd prefer if we didn't use clang-format in scripted diffs. Makes it impossible to reproduce ...",
      "action": false,
      "timestamp": "2017-11-30T19:43:49+00:00"
    },
    {
      "id": "06635d8dcdc544e593c2cefa2569ab45",
      "sender": "gmaxwell",
      "payload": "jtimon: yes true enough, I guess my point was that they're not all bad.",
      "action": false,
      "timestamp": "2017-11-30T19:43:50+00:00"
    },
    {
      "id": "70b7bf9837c942caaffa090d2e98a8da",
      "sender": "wumpus",
      "payload": "kanzure: yes",
      "action": false,
      "timestamp": "2017-11-30T19:43:52+00:00"
    },
    {
      "id": "9eb848a6332a4374b691ca6818e0dc7b",
      "sender": "jonasschnelli",
      "payload": "kanzure: yes",
      "action": false,
      "timestamp": "2017-11-30T19:43:53+00:00"
    },
    {
      "id": "1d120cb7a2b049c3b47c59d11d3dd910",
      "sender": "wumpus",
      "payload": "contrib/devtools/clang-format-diff.py",
      "action": false,
      "timestamp": "2017-11-30T19:44:15+00:00"
    },
    {
      "id": "8141c88779714b7c977507e36c1d609d",
      "sender": "wumpus",
      "payload": "src/.clang-format",
      "action": false,
      "timestamp": "2017-11-30T19:44:15+00:00"
    },
    {
      "id": "e8722013dac5435a9c782d5cec1b0f5f",
      "sender": "kanzure",
      "payload": "ah thanks.",
      "action": false,
      "timestamp": "2017-11-30T19:44:25+00:00"
    },
    {
      "id": "e41e49a9661a4836b38ae32ebb150150",
      "sender": "cfields",
      "payload": "i assume it'd be possible to setup a git alias that shows the current diff, diff'd against the clang-format result",
      "action": false,
      "timestamp": "2017-11-30T19:44:39+00:00"
    },
    {
      "id": "606334a2e5d5417282d56b3de11acadf",
      "sender": "cfields",
      "payload": "plays around",
      "action": true,
      "timestamp": "2017-11-30T19:44:43+00:00"
    },
    {
      "id": "f1e85c1b662b4a32a36a33d39534840a",
      "sender": "jonasschnelli",
      "payload": "MarcoFalke: are you up to write a higher level documentation for the clang-format-diff.py workflow?",
      "action": false,
      "timestamp": "2017-11-30T19:44:44+00:00"
    },
    {
      "id": "d1aa4414b0554a45a9e27adee7480061",
      "sender": "MarcoFalke",
      "payload": "proposed to add '.style.yapf' and hides",
      "action": true,
      "timestamp": "2017-11-30T19:44:49+00:00"
    },
    {
      "id": "cd2295c7bb5d43529c1dcd0ef09335f4",
      "sender": "jtimon",
      "payload": "wumpus: right, that's what I remember being written",
      "action": false,
      "timestamp": "2017-11-30T19:44:56+00:00"
    },
    {
      "id": "dd2e90c3dbba461981a8b4782ef3bfbd",
      "sender": "MarcoFalke",
      "payload": "jonasschnelli: I did",
      "action": false,
      "timestamp": "2017-11-30T19:44:57+00:00"
    },
    {
      "id": "952f6960c27045c199316971939be6d9",
      "sender": "MarcoFalke",
      "payload": "git diff | cfd",
      "action": false,
      "timestamp": "2017-11-30T19:45:06+00:00"
    },
    {
      "id": "fe7a13ebc418401a90405ee66730ea32",
      "sender": "wumpus",
      "payload": "yapf?",
      "action": false,
      "timestamp": "2017-11-30T19:45:18+00:00"
    },
    {
      "id": "f78c13077b964438806886b0afb49f55",
      "sender": "gmaxwell",
      "payload": "general problem with clang format is that it isn't stable across versions, or at least hasn't been historically. In general we don't have a highly consistent enviroment htat people contribute from.",
      "action": false,
      "timestamp": "2017-11-30T19:45:23+00:00"
    },
    {
      "id": "4b32cdc6c02845388d77975c105f6485",
      "sender": "ryanofsky",
      "payload": "fwiw, i am not bothered by nits whatsoever. at the very least they mean somebody is actually looking at my pr. i also use clang-format and clang-format diff all the time",
      "action": false,
      "timestamp": "2017-11-30T19:45:29+00:00"
    },
    {
      "id": "d4741fe172044c378e196f5a98d59414",
      "sender": "wumpus",
      "payload": "gmaxwell: right, that has always bothered about it",
      "action": false,
      "timestamp": "2017-11-30T19:45:36+00:00"
    },
    {
      "id": "bc2a1dd89a6a49ac9c5ce8cfc946cd5d",
      "sender": "jonasschnelli",
      "payload": "gmaxwell: yeah. that",
      "action": false,
      "timestamp": "2017-11-30T19:45:37+00:00"
    },
    {
      "id": "6c948d64946a441dbd75791cbbf7ffd7",
      "sender": "MarcoFalke",
      "payload": "also what gmaxwell said",
      "action": false,
      "timestamp": "2017-11-30T19:45:39+00:00"
    },
    {
      "id": "e2a51d6042504c3d9713cfb2f5ebdabc",
      "sender": "MarcoFalke",
      "payload": "wumpus: The same thing for python",
      "action": false,
      "timestamp": "2017-11-30T19:45:51+00:00"
    },
    {
      "id": "e4f69e07ef4945478ba7f26a2c5bae51",
      "sender": "jtimon",
      "payload": "well, the version can be fixed on travis, though",
      "action": false,
      "timestamp": "2017-11-30T19:45:59+00:00"
    },
    {
      "id": "e4736b0b69914570a8b2135725b09858",
      "sender": "wumpus",
      "payload": "MarcoFalke: that's not very shocking",
      "action": false,
      "timestamp": "2017-11-30T19:46:02+00:00"
    },
    {
      "id": "d20bfc73a2814c91ba994053b054cbae",
      "sender": "jonasschnelli",
      "payload": "ryanofsky: nits should still remain... just not stuff that computers can find and are covered by our code styling rules",
      "action": false,
      "timestamp": "2017-11-30T19:46:21+00:00"
    },
    {
      "id": "573463df393f4e98a44ba1c7ea3cbeef",
      "sender": "sipa",
      "payload": "jtimon: right, but that makes travis effectively part of your workflow, which is unfortunate",
      "action": false,
      "timestamp": "2017-11-30T19:46:23+00:00"
    },
    {
      "id": "b8b53f04ef0442ae8a2dfaba3f950f12",
      "sender": "wumpus",
      "payload": "you need to be able to run any checks that travis does locally",
      "action": false,
      "timestamp": "2017-11-30T19:46:42+00:00"
    },
    {
      "id": "b76a171b0d7945e58a692f78cc3f48e7",
      "sender": "sipa",
      "payload": "jtimon: as in, you won't really know a PR is acceptable before pushing and waiting for travis",
      "action": false,
      "timestamp": "2017-11-30T19:46:43+00:00"
    },
    {
      "id": "937cd6fb339b4cc390df6b52886399cf",
      "sender": "gmaxwell",
      "payload": "We could give developers a VM image or equivilent, but it's asking a lot.",
      "action": false,
      "timestamp": "2017-11-30T19:46:47+00:00"
    },
    {
      "id": "dc2c94b8f59a414e9bed8af2ce995f3e",
      "sender": "jtimon",
      "payload": "sipa: right, or the concrete version if I want to run it locally first",
      "action": false,
      "timestamp": "2017-11-30T19:46:48+00:00"
    },
    {
      "id": "c2210df12e7c4d2285bfa7affbb99644",
      "sender": "luke-jr",
      "payload": "Travis is slow, too",
      "action": false,
      "timestamp": "2017-11-30T19:47:19+00:00"
    },
    {
      "id": "404ff69f78fb45b5938634737a9bae32",
      "sender": "sipa",
      "payload": "maybe someone should investigate how stable clang-format/ clang-tidy/ iwyu/ ... are",
      "action": false,
      "timestamp": "2017-11-30T19:47:30+00:00"
    },
    {
      "id": "cd49aa24c97e4d3f9acfd26de0d522e7",
      "sender": "morcos",
      "payload": "my issue with the nits on PR's is it leads to sloppier review.  When nits get fixed and potentially rebased.  Prior reviewers can get sloppy about assuming that the final version is well reviewed, especially if its happened repeatedly",
      "action": false,
      "timestamp": "2017-11-30T19:47:33+00:00"
    },
    {
      "id": "9b6e887038ca4e7885daf996cd2a8b2e",
      "sender": "jonasschnelli",
      "payload": "Could we not do a check with clang-format-diff.py during \"make\" and at least place a bold warning (or even refuse to compile *duck*)",
      "action": false,
      "timestamp": "2017-11-30T19:47:39+00:00"
    },
    {
      "id": "20c2986785f74832a93a3660e30ff0c8",
      "sender": "achow101",
      "payload": "jonasschnelli: it would blow up on existing code",
      "action": false,
      "timestamp": "2017-11-30T19:48:00+00:00"
    },
    {
      "id": "cdb86e0ec4cb4c46a503889fd8787fb4",
      "sender": "sipa",
      "payload": "jonasschnelli: the problem is that different clang-format versions say different things",
      "action": false,
      "timestamp": "2017-11-30T19:48:07+00:00"
    },
    {
      "id": "675a8f9091414e449d57aeb60cbb6f1c",
      "sender": "jonasschnelli",
      "payload": "sipa: significant?",
      "action": false,
      "timestamp": "2017-11-30T19:48:20+00:00"
    },
    {
      "id": "c70c48a49d524f858516956cdde6375f",
      "sender": "sipa",
      "payload": "jonasschnelli: irrelevant, i think",
      "action": false,
      "timestamp": "2017-11-30T19:48:26+00:00"
    },
    {
      "id": "bcaca4f27f1448939672583b0de63b13",
      "sender": "wumpus",
      "payload": "jonasschnelli: 'make lint' would be nice",
      "action": false,
      "timestamp": "2017-11-30T19:48:29+00:00"
    },
    {
      "id": "d1de88b088eb47658b124fb7da7bebe7",
      "sender": "sipa",
      "payload": "and making something work on your own system, and then seeing travis complain about the exact same things but requiring something else would be pretty demotivating",
      "action": false,
      "timestamp": "2017-11-30T19:48:45+00:00"
    },
    {
      "id": "900d2aa753e04e9fb7ba812ecb55192d",
      "sender": "wumpus",
      "payload": "jonasschnelli: that would just run all the checks, for C/C++, for python, for whitespace in docs, etc",
      "action": false,
      "timestamp": "2017-11-30T19:48:49+00:00"
    },
    {
      "id": "d8705052fe604e9e896adf6abd691530",
      "sender": "cfields",
      "payload": "wumpus: +1",
      "action": false,
      "timestamp": "2017-11-30T19:48:49+00:00"
    },
    {
      "id": "3095b5943eb14ad38cc81f8b249ec1d8",
      "sender": "gmaxwell",
      "payload": "morcos: that is more a problem with the intermixing of nits and serious review, and also how github handles tracking comments (them magically vanishing when the code changes)",
      "action": false,
      "timestamp": "2017-11-30T19:48:57+00:00"
    },
    {
      "id": "eed105ef0ab54b5197139d945c665d7e",
      "sender": "jonasschnelli",
      "payload": "wumpus: Maybe it should by bypassable, but the novice contributor should get warned if he uses invalid code-styling",
      "action": false,
      "timestamp": "2017-11-30T19:49:08+00:00"
    },
    {
      "id": "0f708eb6d1f1486098da35b3d749ce9d",
      "sender": "sipa",
      "payload": "gmaxwell: that's no longer true with the 'review' function",
      "action": false,
      "timestamp": "2017-11-30T19:49:15+00:00"
    },
    {
      "id": "0af407c1076e43c2bc046f3bc218bde2",
      "sender": "gmaxwell",
      "payload": "Obviously we should just pull clang into our code tree and ship it too. :P",
      "action": false,
      "timestamp": "2017-11-30T19:49:28+00:00"
    },
    {
      "id": "3f1069964f6047f18d6c7596feb534cd",
      "sender": "sipa",
      "payload": "you can see all former review comments, and the code they apply on",
      "action": false,
      "timestamp": "2017-11-30T19:49:28+00:00"
    },
    {
      "id": "adabb79ce7ca4dc794d70bcfe690c8df",
      "sender": "wumpus",
      "payload": "jonasschnelli: well, travis and the person that merge can also check",
      "action": false,
      "timestamp": "2017-11-30T19:49:30+00:00"
    },
    {
      "id": "6a35b5ea92534913ba6601eb56ed1de4",
      "sender": "jtimon",
      "payload": "sipa: oh, that's what is for...",
      "action": false,
      "timestamp": "2017-11-30T19:49:37+00:00"
    },
    {
      "id": "f0f846891f9d4063bfdc0fc2c3d9b215",
      "sender": "instagibbs",
      "payload": "sipa, interesting, more reason to use it",
      "action": false,
      "timestamp": "2017-11-30T19:49:37+00:00"
    },
    {
      "id": "2c4ec24e408e4156a58db6be500917f7",
      "sender": "wumpus",
      "payload": "jonasschnelli: it just should be possible to do it locally too",
      "action": false,
      "timestamp": "2017-11-30T19:49:38+00:00"
    },
    {
      "id": "ab418748fda34a9197aa1e4414e132cf",
      "sender": "morcos",
      "payload": "gmaxwell: yes, i'd argue that once serious review stars, nits should not be squashed, but should be left as a cleanup commit at the end.  (at least of some kinds)",
      "action": false,
      "timestamp": "2017-11-30T19:49:49+00:00"
    },
    {
      "id": "8d7143376a0241869bc4a364a348fa41",
      "sender": "sipa",
      "payload": "jtimon: yes, you should use it :)",
      "action": false,
      "timestamp": "2017-11-30T19:49:51+00:00"
    },
    {
      "id": "04909096c9e84aa8a02840b3263f91bf",
      "sender": "wumpus",
      "payload": "jonasschnelli: I'm not so much concerned with forcing people to run it but making it easy",
      "action": false,
      "timestamp": "2017-11-30T19:49:54+00:00"
    },
    {
      "id": "c7a399123d9d4f3b88b6808ce18ce969",
      "sender": "jonasschnelli",
      "payload": "wumpus: yes. Travis code style check is not what we want in the first place (it helps,.. but you want to catch it before)",
      "action": false,
      "timestamp": "2017-11-30T19:50:08+00:00"
    },
    {
      "id": "412a6b957a3d4b9db41987a301b79cab",
      "sender": "wumpus",
      "payload": "jonasschnelli: well if you don't do it before, travis will stop you and you need to wait longer, simple as that :)",
      "action": false,
      "timestamp": "2017-11-30T19:50:33+00:00"
    },
    {
      "id": "d1ba2a795eb64e929a5a518f88fedb0b",
      "sender": "sipa",
      "payload": "something i have noticed is that sometimes 'nit' reviews start on PRs which are far from certain they'll be even concept ACKed",
      "action": false,
      "timestamp": "2017-11-30T19:50:33+00:00"
    },
    {
      "id": "a2c4a153b6c14fbe93bdbe1b8e4ffffc",
      "sender": "instagibbs",
      "payload": "morcos, we might need to have guidelines for \"ACK lockin\"",
      "action": false,
      "timestamp": "2017-11-30T19:50:39+00:00"
    },
    {
      "id": "c774111c4d434e5997b342129292576c",
      "sender": "jonasschnelli",
      "payload": "Okay... I'll have a look at lint and something simple within the make-process",
      "action": false,
      "timestamp": "2017-11-30T19:50:44+00:00"
    },
    {
      "id": "bb278104c5364739b3a062bd45667804",
      "sender": "jonasschnelli",
      "payload": "sipa: that also",
      "action": false,
      "timestamp": "2017-11-30T19:50:46+00:00"
    },
    {
      "id": "92e1eb702b2641e89e16c304212a049f",
      "sender": "cfields",
      "payload": "morcos: i have no problem with squashing nits as the pre/post-squash can be diff'd locally. It's rebasing to master that makes re-review a challenge imo.",
      "action": false,
      "timestamp": "2017-11-30T19:50:52+00:00"
    },
    {
      "id": "3c4409f1120141d2871f1a5edb8292f1",
      "sender": "wumpus",
      "payload": "sipa: yeah... then it adds even more nosie",
      "action": false,
      "timestamp": "2017-11-30T19:51:05+00:00"
    },
    {
      "id": "326357dd509b4183a1a0e098ad8d5f77",
      "sender": "instagibbs",
      "payload": "cfields, can be diffed if you locally checked",
      "action": false,
      "timestamp": "2017-11-30T19:51:07+00:00"
    },
    {
      "id": "67136a9ddee34ba1912c4dfc017fe343",
      "sender": "sipa",
      "payload": "cfields: right, i try to avoid rebasing, but i pretty liberally squash nits",
      "action": false,
      "timestamp": "2017-11-30T19:51:20+00:00"
    },
    {
      "id": "63a733cc68704ddbbbf49e94d4b657a1",
      "sender": "instagibbs",
      "payload": "many times im just reading off of github(lazy)",
      "action": false,
      "timestamp": "2017-11-30T19:51:21+00:00"
    },
    {
      "id": "851e85ebd2ae4186be9060f272dbaaa8",
      "sender": "sipa",
      "payload": "cfields: and i don't mind others doing that too, except for very complicated PRs",
      "action": false,
      "timestamp": "2017-11-30T19:51:36+00:00"
    },
    {
      "id": "de2643a1ecd84c88b36178f8099efb0e",
      "sender": "instagibbs",
      "payload": "unless it's a particularly intense series of commits",
      "action": false,
      "timestamp": "2017-11-30T19:51:38+00:00"
    },
    {
      "id": "63d88503451745c690e4b80c3916f4e8",
      "sender": "achow101",
      "payload": "instagibbs: that kind of breaks when there's lots of merge conflicts",
      "action": false,
      "timestamp": "2017-11-30T19:51:41+00:00"
    },
    {
      "id": "b407a60ceb77451682a2b9442dae1044",
      "sender": "wumpus",
      "payload": "sometimes I wish 'git fetch' could fetch by commit id to fetch individual commits, unfortunately that doesn't work",
      "action": false,
      "timestamp": "2017-11-30T19:51:51+00:00"
    },
    {
      "id": "126007a9ace7410f8727dac65af89aca",
      "sender": "cfields",
      "payload": "pretty sure github can show the diff from a squash a well, you just have to come up with the url for it",
      "action": false,
      "timestamp": "2017-11-30T19:52:00+00:00"
    },
    {
      "id": "5e02f9258fcc4de89f9ce188d4a26728",
      "sender": "gmaxwell",
      "payload": "well we have contributors who don't feel comfortable (or just don't have the expirence) to do much other than nit reviews. Their contributions are usually pretty helpful and I wouldn't want to ask most of them to stop. We could perhaps ask them to wait a day on new PRs so that there is at least time for Concept NAKs to show up first.",
      "action": false,
      "timestamp": "2017-11-30T19:52:03+00:00"
    },
    {
      "id": "f2d527ffa94c44038f52c7b9423198e7",
      "sender": "wumpus",
      "payload": "it can only be used with branch names",
      "action": false,
      "timestamp": "2017-11-30T19:52:05+00:00"
    },
    {
      "id": "4849062e54fc47a4bf833103d9762129",
      "sender": "wumpus",
      "payload": "github can give you the patch for an arbitrary commit id though, but it's kind of annoying to do automatically",
      "action": false,
      "timestamp": "2017-11-30T19:52:38+00:00"
    },
    {
      "id": "605f1ae3369c4b7a87cae5e6c5248c56",
      "sender": "gmaxwell",
      "payload": "it is a little sad for someone to show up with a change, and then have two cycles on trivial changes before someone more expirenced comes in and says no-way-because-x.",
      "action": false,
      "timestamp": "2017-11-30T19:52:54+00:00"
    },
    {
      "id": "8694400d2d0e4feca42b589ce5c4239e",
      "sender": "wumpus",
      "payload": "gmaxwell: yeah...",
      "action": false,
      "timestamp": "2017-11-30T19:53:02+00:00"
    },
    {
      "id": "448fa5c328ca49d8bb26defc46232ff5",
      "sender": "aj",
      "payload": "gmaxwell: might be helpful to tag PRs as looking for concept acks vs looking for nits and style vs looking for tested acks and merging?",
      "action": false,
      "timestamp": "2017-11-30T19:53:04+00:00"
    },
    {
      "id": "a5c14af89cae4dfd91cd2f34fc4ffc97",
      "sender": "MarcoFalke",
      "payload": "wumpus: We could set up a bot to create branches for each utACK that is posted to gh",
      "action": false,
      "timestamp": "2017-11-30T19:53:08+00:00"
    },
    {
      "id": "130ec8972f2f461c89b68091d7b25596",
      "sender": "instagibbs",
      "payload": "not nitting a PR that doens't have any ACKs of any kind seems like a decent rule",
      "action": false,
      "timestamp": "2017-11-30T19:53:13+00:00"
    },
    {
      "id": "6276ae615f064ccca106cca114d81761",
      "sender": "MarcoFalke",
      "payload": "on a separate repo that is",
      "action": false,
      "timestamp": "2017-11-30T19:53:16+00:00"
    },
    {
      "id": "f0bc3cd24b504c4dbc0a32d7f54c209c",
      "sender": "wumpus",
      "payload": "MarcoFalke: that'd be kind of nice",
      "action": false,
      "timestamp": "2017-11-30T19:53:23+00:00"
    },
    {
      "id": "023889e4adf845e5b9c951678736f02e",
      "sender": "jtimon",
      "payload": "cfields: at the same time rebasing is often good, for example, #8994 could unexpectedly start failing tests even if no new conflicts appear and github says it's all ready to merge it",
      "action": false,
      "timestamp": "2017-11-30T19:53:28+00:00"
    },
    {
      "id": "bd1b4946b48641e09d5332c1ce74fba8",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon \u00c3\u0082\u00c2\u00b7 Pull Request #8994 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2017-11-30T19:53:32+00:00"
    },
    {
      "id": "9446dba66c12455c8a086e4767869911",
      "sender": "wumpus",
      "payload": "rebasing is often necessary",
      "action": false,
      "timestamp": "2017-11-30T19:53:48+00:00"
    },
    {
      "id": "6b4a0c49179847ecb37b33ac9f83386d",
      "sender": "jonasschnelli",
      "payload": "Yes. What aj says. The things is just, unenforced user rules tend to get ignored. :)",
      "action": false,
      "timestamp": "2017-11-30T19:53:52+00:00"
    },
    {
      "id": "ce8a36cc7ef44213a4cb11abb1806f01",
      "sender": "Provoostenator",
      "payload": "gmaxwell: as long as you learn something from the nits, it's not the end of the world having them concept-nacked later, imo",
      "action": false,
      "timestamp": "2017-11-30T19:53:54+00:00"
    },
    {
      "id": "8ecb5e7098e34e8497e8da5627394cda",
      "sender": "MarcoFalke",
      "payload": "then `git fetch bitcoin_reviewed_commits` will get you all the reviews",
      "action": false,
      "timestamp": "2017-11-30T19:54:00+00:00"
    },
    {
      "id": "8315ba5bf72c4d6e9def1808be30fd29",
      "sender": "gmaxwell",
      "payload": "sometimes you don't know what you're going to do when you read it.. there are certantly PRs where I read them and then come away with no real opinions on it other than \"you missed some braces here and there\"",
      "action": false,
      "timestamp": "2017-11-30T19:54:01+00:00"
    },
    {
      "id": "3d5628e566864686928be1634228343e",
      "sender": "wumpus",
      "payload": "some files are hotspots and generate a lot of merge conflicts",
      "action": false,
      "timestamp": "2017-11-30T19:54:13+00:00"
    },
    {
      "id": "81aad239d767480dbefd6d08392fe27c",
      "sender": "wumpus",
      "payload": "although it's better now that main.cpp is dead",
      "action": false,
      "timestamp": "2017-11-30T19:54:28+00:00"
    },
    {
      "id": "f4d1193bb2574d1fa40745fffedb62cf",
      "sender": "gmaxwell",
      "payload": "Provoostenator: no, but some people find it demotivating; 'they made me do more work then threw it out'. :)",
      "action": false,
      "timestamp": "2017-11-30T19:54:32+00:00"
    },
    {
      "id": "ff37119f1768444492a12683a4997720",
      "sender": "Provoostenator",
      "payload": "True",
      "action": false,
      "timestamp": "2017-11-30T19:54:52+00:00"
    },
    {
      "id": "a929abda021242ef8cf91ba04529ce77",
      "sender": "wumpus",
      "payload": "Provoostenator: these are not the kind of nits that help you learn btter programming :)",
      "action": false,
      "timestamp": "2017-11-30T19:55:10+00:00"
    },
    {
      "id": "50080cd6b6a54fa9b7454b49ffef381b",
      "sender": "gmaxwell",
      "payload": "Provoostenator: that can be resolved with a better perspective; bitcoin development is a distributed process, your effort is your contribution not the fact that it was merged, etc.",
      "action": false,
      "timestamp": "2017-11-30T19:55:30+00:00"
    },
    {
      "id": "7d74b9301bed42f2bfa5b178ac9599d5",
      "sender": "jonasschnelli",
      "payload": "gmaxwell: indeed",
      "action": false,
      "timestamp": "2017-11-30T19:55:48+00:00"
    },
    {
      "id": "dafbae8ed5ae4091bb535032ade0324e",
      "sender": "gmaxwell",
      "payload": "Provoostenator: but we can't really send every new contributor to a zen mastery class before they contribute.",
      "action": false,
      "timestamp": "2017-11-30T19:55:52+00:00"
    },
    {
      "id": "f2157c0283ef4beca95a8241ff10e43d",
      "sender": "Provoostenator",
      "payload": "Well, we could... but that would be flagged as spam :-)",
      "action": false,
      "timestamp": "2017-11-30T19:56:10+00:00"
    },
    {
      "id": "ac27a285822c4feebcc53099b7d8d48e",
      "sender": "wumpus",
      "payload": "Provoostenator: if it's a nit like 'it's better to use build in function X' or 'the loop can be more efficiently written like this' then I think it's great",
      "action": false,
      "timestamp": "2017-11-30T19:56:11+00:00"
    },
    {
      "id": "0fd88195874e43148c38b804a4378186",
      "sender": "cfields",
      "payload": "a slightly different work-flow might be: create a \"fixed nits\" commit on top of the branch, and squash all nits into that as they arise. Then merge that in without squashing.",
      "action": false,
      "timestamp": "2017-11-30T19:57:17+00:00"
    },
    {
      "id": "798405d85e314d2985d89fb7e2ce4acd",
      "sender": "cfields",
      "payload": "it makes for ugly commits getting merged in, but avoids the re-review after squash-for-nits issue.",
      "action": false,
      "timestamp": "2017-11-30T19:57:40+00:00"
    },
    {
      "id": "c89b4876cedc42f5ad1cbef4ab48416d",
      "sender": "Provoostenator",
      "payload": "So far I find most of the nits I received useful. They either taught me to look up coding practices, or motivated me to figure out how to run a linter.",
      "action": false,
      "timestamp": "2017-11-30T19:57:41+00:00"
    },
    {
      "id": "daecb90d0f1a41f5812d3c77cf350b37",
      "sender": "gmaxwell",
      "payload": "a challenge with that is that we do get new contributions from people with zero git expirence.",
      "action": false,
      "timestamp": "2017-11-30T19:57:44+00:00"
    },
    {
      "id": "70e5b8a4e29a426fb9cdc0f01ce9d39f",
      "sender": "sipa",
      "payload": "who think they need to open a new PR to fix a nit :)",
      "action": false,
      "timestamp": "2017-11-30T19:58:10+00:00"
    },
    {
      "id": "f641d8681eff4307a02965973ae8c7a4",
      "sender": "wumpus",
      "payload": "the git basics stuff is explained pretty well in the contributing doc nowadays",
      "action": false,
      "timestamp": "2017-11-30T19:58:10+00:00"
    },
    {
      "id": "78c46abb6430486e93b3467587bb978b",
      "sender": "aj",
      "payload": "do people think the +1, +0, -1, concept nak/ack thing from https://github.com/bitcoin/bitcoin/pull/11426#issuecomment-334091207 is good btw?",
      "action": false,
      "timestamp": "2017-11-30T19:58:27+00:00"
    },
    {
      "id": "ecbfd42b68304107a5cb0717d26192d3",
      "sender": "wumpus",
      "payload": "I've not gotten the question on how to squash a commit for a long time now",
      "action": false,
      "timestamp": "2017-11-30T19:58:29+00:00"
    },
    {
      "id": "d2470ba0b56d46a4b51fe19598c4a553",
      "sender": "gmaxwell",
      "payload": "wumpus: I've noticed that, I wondered why.",
      "action": false,
      "timestamp": "2017-11-30T19:58:41+00:00"
    },
    {
      "id": "0a5a2fdc76a44b83aac425af4fe74f84",
      "sender": "instagibbs",
      "payload": "oh that's where -0 came from",
      "action": false,
      "timestamp": "2017-11-30T19:59:03+00:00"
    },
    {
      "id": "1bb94bd9bcf246b88c1b2037b99a431e",
      "sender": "Provoostenator",
      "payload": "Some projects make a giant squashed merge commit and just point to the original PR (e.g. AngularJS), but that's not ideal for other reasons.",
      "action": false,
      "timestamp": "2017-11-30T19:59:03+00:00"
    },
    {
      "id": "ebee33f57b914f8ab284d6e0d9597b7a",
      "sender": "BlueMatt",
      "payload": "aj: yes, I'm a fan",
      "action": false,
      "timestamp": "2017-11-30T19:59:33+00:00"
    },
    {
      "id": "4893642a11c44f8fafc545a81d820b59",
      "sender": "wumpus",
      "payload": "Provoostenator: we only want to encourage squashing when it's iterative changes, not atomic separate changes",
      "action": false,
      "timestamp": "2017-11-30T19:59:37+00:00"
    },
    {
      "id": "aa9e6605446847cca7e7296b04605d6b",
      "sender": "jtimon",
      "payload": "aj: I would have preffered that BlueMatt had maintained a nack but answered my questions/rebute, to be honest",
      "action": false,
      "timestamp": "2017-11-30T19:59:40+00:00"
    },
    {
      "id": "ed92d1fff598418d98da2e8aff4b79bd",
      "sender": "BlueMatt",
      "payload": "specifically caues we end up with lots of need for concept review",
      "action": false,
      "timestamp": "2017-11-30T19:59:42+00:00"
    },
    {
      "id": "ff3e9c18ea574e3e958dec4ef08ad48e",
      "sender": "BlueMatt",
      "payload": "we have lots of prs where lots of folks are +0/-0 and dont think its worth review",
      "action": false,
      "timestamp": "2017-11-30T20:00:13+00:00"
    },
    {
      "id": "a5cca7e6e2eb428d8fe9bfac7cd3a314",
      "sender": "BlueMatt",
      "payload": "but they sit open for months",
      "action": false,
      "timestamp": "2017-11-30T20:00:16+00:00"
    },
    {
      "id": "18f6ec23bb714f5fb1ca806e5fa5cce7",
      "sender": "BlueMatt",
      "payload": "which is bad",
      "action": false,
      "timestamp": "2017-11-30T20:00:17+00:00"
    },
    {
      "id": "e88dfb49ed974d95ad02e025927b6a0c",
      "sender": "MarcoFalke",
      "payload": "+0 on end meeting",
      "action": false,
      "timestamp": "2017-11-30T20:00:24+00:00"
    },
    {
      "id": "328079f1a610486f801f8257bdc77422",
      "sender": "BlueMatt",
      "payload": "+1",
      "action": false,
      "timestamp": "2017-11-30T20:00:29+00:00"
    },
    {
      "id": "9168ef775a3741828e4c4ec418b21122",
      "sender": "instagibbs",
      "payload": "-0",
      "action": false,
      "timestamp": "2017-11-30T20:00:29+00:00"
    },
    {
      "id": "956e25df0e3a4bd394125149c8aeb5a1",
      "sender": "wumpus",
      "payload": "oh, it's that time already",
      "action": false,
      "timestamp": "2017-11-30T20:00:31+00:00"
    },
    {
      "id": "4702a5db2683485eafb51ab0c53b59d0",
      "sender": "wumpus",
      "payload": "#endmeeting",
      "action": false,
      "timestamp": "2017-11-30T20:00:33+00:00"
    }
  ],
  "events": [
    {
      "event_type": "START_MEETING",
      "message": {
        "id": "5a5587551f644c148a179f39ce9ceb49",
        "sender": "wumpus",
        "payload": "#startmeeting",
        "action": false,
        "timestamp": "2017-11-30T19:00:03+00:00"
      },
      "operand": null,
      "id": "5a5587551f644c148a179f39ce9ceb49",
      "timestamp": "2017-11-30T19:00:03+00:00"
    },
    {
      "event_type": "TOPIC",
      "message": {
        "id": "bdf4370f984040b2b6926110944ba5d0",
        "sender": "wumpus",
        "payload": "#topic high priority for review",
        "action": false,
        "timestamp": "2017-11-30T19:00:59+00:00"
      },
      "operand": "high priority for review",
      "id": "bdf4370f984040b2b6926110944ba5d0",
      "timestamp": "2017-11-30T19:00:59+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "523c93e357ff429ebd26c853874e2780",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/10279 | Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces by TheBlueMatt \u00c3\u0082\u00c2\u00b7 Pull Request #10279 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:04:15+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/10279",
      "id": "523c93e357ff429ebd26c853874e2780",
      "timestamp": "2017-11-30T19:04:15+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "058395c87c484650b3edea98443e02d4",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #11403 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:06:30+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/11403",
      "id": "058395c87c484650b3edea98443e02d4",
      "timestamp": "2017-11-30T19:06:30+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "d696a13f82e747e1a4f175c6b1561ff9",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa \u00c3\u0082\u00c2\u00b7 Pull Request #11403 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:08:34+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/11403",
      "id": "d696a13f82e747e1a4f175c6b1561ff9",
      "timestamp": "2017-11-30T19:08:34+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "9cbcde1d9f74475eac03341e19c51df8",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon \u00c3\u0082\u00c2\u00b7 Pull Request #8994 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:10:02+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/8994",
      "id": "9cbcde1d9f74475eac03341e19c51df8",
      "timestamp": "2017-11-30T19:10:02+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "d616789536014e81af875e4a0b0661f5",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/10757 | RPC: Introduce getblockstats to plot things by jtimon \u00c3\u0082\u00c2\u00b7 Pull Request #10757 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:10:07+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/10757",
      "id": "d616789536014e81af875e4a0b0661f5",
      "timestamp": "2017-11-30T19:10:07+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "b169e8be93394e8ab09322403eb9d494",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/10583 | [RPC] Split part of validateaddress into getaddressinfo by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #10583 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:21:10+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/10583",
      "id": "b169e8be93394e8ab09322403eb9d494",
      "timestamp": "2017-11-30T19:21:10+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "d6dcb7fa0b58485398243d8d1afd96d9",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/10579 | [RPC] Split signrawtransaction into wallet and non-wallet RPC command by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #10579 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:21:14+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/10579",
      "id": "d6dcb7fa0b58485398243d8d1afd96d9",
      "timestamp": "2017-11-30T19:21:14+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "1369bc6ae7fd4cc3812ce9034d715ed2",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/11415 | [RPC] Disallow using addresses in createmultisig by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #11415 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:21:17+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/11415",
      "id": "1369bc6ae7fd4cc3812ce9034d715ed2",
      "timestamp": "2017-11-30T19:21:17+00:00"
    },
    {
      "event_type": "TOPIC",
      "message": {
        "id": "b29d1a93772f477fb0976a9c8790153d",
        "sender": "wumpus",
        "payload": "#topic codesigning",
        "action": false,
        "timestamp": "2017-11-30T19:22:09+00:00"
      },
      "operand": "codesigning",
      "id": "b29d1a93772f477fb0976a9c8790153d",
      "timestamp": "2017-11-30T19:22:09+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "da51433b08cd4a29adeff3fbe0ecaad4",
        "sender": "jonasschnelli",
        "payload": "https://github.com/bitcoincore-codesigning-association",
        "action": false,
        "timestamp": "2017-11-30T19:22:28+00:00"
      },
      "operand": "https://github.com/bitcoincore-codesigning-association",
      "id": "da51433b08cd4a29adeff3fbe0ecaad4",
      "timestamp": "2017-11-30T19:22:28+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "33572be694de46ec921276e2bb6cc135",
        "sender": "jonasschnelli",
        "payload": "https://bitcoincorecodesigning.org",
        "action": false,
        "timestamp": "2017-11-30T19:22:45+00:00"
      },
      "operand": "https://bitcoincorecodesigning.org",
      "id": "33572be694de46ec921276e2bb6cc135",
      "timestamp": "2017-11-30T19:22:45+00:00"
    },
    {
      "event_type": "TOPIC",
      "message": {
        "id": "1d76cd765b5345c68e1bbb731fac6fb3",
        "sender": "wumpus",
        "payload": "#topic review nit reduction",
        "action": false,
        "timestamp": "2017-11-30T19:34:48+00:00"
      },
      "operand": "review nit reduction",
      "id": "1d76cd765b5345c68e1bbb731fac6fb3",
      "timestamp": "2017-11-30T19:34:48+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "220a0cfb6afe4accb14c5ba3a161d20b",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/11699 | [travis-ci] Only run linters on Pull Requests by jnewbery \u00c3\u0082\u00c2\u00b7 Pull Request #11699 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:39:10+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/11699",
      "id": "220a0cfb6afe4accb14c5ba3a161d20b",
      "timestamp": "2017-11-30T19:39:10+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "bd1b4946b48641e09d5332c1ce74fba8",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon \u00c3\u0082\u00c2\u00b7 Pull Request #8994 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2017-11-30T19:53:32+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/8994",
      "id": "bd1b4946b48641e09d5332c1ce74fba8",
      "timestamp": "2017-11-30T19:53:32+00:00"
    },
    {
      "event_type": "END_MEETING",
      "message": {
        "id": "4702a5db2683485eafb51ab0c53b59d0",
        "sender": "wumpus",
        "payload": "#endmeeting",
        "action": false,
        "timestamp": "2017-11-30T20:00:33+00:00"
      },
      "operand": null,
      "id": "4702a5db2683485eafb51ab0c53b59d0",
      "timestamp": "2017-11-30T20:00:33+00:00"
    }
  ],
  "aliases": {},
  "vote_in_progress": false,
  "motion_index": null
}