Undo unintended description update

source/platforms/gitlab/_tests/fixtures/getMergeRequestChanges.json

@@ -10,7 +10,7 @@
       "iid": 27117,
       "project_id": 13083,
       "title": "Stable reviewer roulette",
-      "description": "Change reviewer roulette to always pick the same reviewers for the same\nbranch name. We do this by:\n\n1. Making the branch name 'canonical' across CE and EE by stripping a\n   leading 'ce-' or 'ee-' and a trailing '-ce' or '-ee'. If people are\n   following our branch naming guidelines, this should give the same\n   branch name in both repos.\n2. Converting the branch name to a stable integer by taking the integer\n   form of its MD5.\n3. Passing that integer as a seed to Ruby's `Random` class, which 'may\n   be used to ensure repeatable sequences of pseudo-random numbers\n   between different runs of the program' (from the Ruby documentation).\n\nThe upshot is that the same branch name (in CE and EE) should always\npick the same reviewers, and those should be evenly distributed across\nthe set of possible reviewers due to the use of MD5.\n\nAgain, I have a test script:\n\n```ruby\nrequire 'ffaker'\n\nclass Foo\n  include Gitlab::Danger::Helper\nend\n\ndef spin(team, project, category, branch_name)\n  # Strip leading and trailing CE/EE markers\n  canonical_branch_name = branch_name.gsub(/^[ce]e-/, '').gsub(/-[ce]e$/, '')\n  rng = Random.new(Digest::MD5.hexdigest(canonical_branch_name).to_i(16))\n\n  reviewers = team.select { |member| member.reviewer?(project, category) }\n  traintainers = team.select { |member| member.traintainer?(project, category) }\n  maintainers = team.select { |member| member.maintainer?(project, category) }\n\n  # TODO: filter out people who are currently not in the office\n  # https://gitlab.com/gitlab-org/gitlab-ce/issues/57652\n  #\n  # TODO: take CODEOWNERS into account?\n  # https://gitlab.com/gitlab-org/gitlab-ce/issues/57653\n\n  # Make traintainers have triple the chance to be picked as a reviewer\n  reviewer = (reviewers + traintainers + traintainers).sample(random: rng)\n  maintainer = maintainers.sample(random: rng)\n\n  [reviewer.username, maintainer.username]\nend\n\ndef random_branch_name\n  FFaker::Filesystem.file_name\nend\n\nFFaker::Random.seed = 123\nteam = Foo.new.project_team\nresults = Hash.new(0)\n\n10_000.times do\n  reviewer, maintainer = spin(team, 'gitlab-foss', 'backend', random_branch_name)\n\n  results[reviewer] += 1\n  results[maintainer] += 1\nend\n\nresults.sort_by(&:last).reverse.each do |username, picked|\n  puts \"#{username}: #{picked}\"\nend; nil\n```\n\nThis should output the same for you as it does for me, because we seed the branch names too!\n\n```\ndzaporozhets: 799\nmkozono: 797\ngrzesiek: 794\ngodfat: 793\nDouweM: 788\nnick.thomas: 773\ntkuah: 764\nstanhu: 761\nayufan: 759\njprovaznik: 758\njameslopez: 757\ndbalexandre: 754\nsmcgivern: 751\nsplattael: 744\nashmckenzie: 741\nrspeicher: 739\njarka: 738\nrymai: 735\nreprazent: 729\nmayra-cabrera: 713\nmdelaossa: 273\nrdavila: 269\nvsizov: 260\ntheoretick: 257\noswaldo: 255\nifarkas: 255\nbrytannia: 248\nengwan: 248\nfelipe_artur: 247\nrpereira2: 243\ntoon: 239\nrossfuhrman: 236\njamedjo: 235\nfjsanpedro: 229\nmatteeyah: 226\nbrodock: 226\nvzagorodny: 222\nzj: 220\nmarkglenfletcher: 219\ndosuken123: 206\n```\n\nCloses https://gitlab.com/gitlab-org/gitlab-ce/issues/57766.",
+      "description": "Change reviewer roulette to always pick the same reviewers for the same\nbranch name. We do this by:\n\n1. Making the branch name 'canonical' across CE and EE by stripping a\n   leading 'ce-' or 'ee-' and a trailing '-ce' or '-ee'. If people are\n   following our branch naming guidelines, this should give the same\n   branch name in both repos.\n2. Converting the branch name to a stable integer by taking the integer\n   form of its MD5.\n3. Passing that integer as a seed to Ruby's `Random` class, which 'may\n   be used to ensure repeatable sequences of pseudo-random numbers\n   between different runs of the program' (from the Ruby documentation).\n\nThe upshot is that the same branch name (in CE and EE) should always\npick the same reviewers, and those should be evenly distributed across\nthe set of possible reviewers due to the use of MD5.\n\nAgain, I have a test script:\n\n```ruby\nrequire 'ffaker'\n\nclass Foo\n  include Gitlab::Danger::Helper\nend\n\ndef spin(team, project, category, branch_name)\n  # Strip leading and trailing CE/EE markers\n  canonical_branch_name = branch_name.gsub(/^[ce]e-/, '').gsub(/-[ce]e$/, '')\n  rng = Random.new(Digest::MD5.hexdigest(canonical_branch_name).to_i(16))\n\n  reviewers = team.select { |member| member.reviewer?(project, category) }\n  traintainers = team.select { |member| member.traintainer?(project, category) }\n  maintainers = team.select { |member| member.maintainer?(project, category) }\n\n  # TODO: filter out people who are currently not in the office\n  # https://gitlab.com/gitlab-org/gitlab-ce/issues/57652\n  #\n  # TODO: take CODEOWNERS into account?\n  # https://gitlab.com/gitlab-org/gitlab-ce/issues/57653\n\n  # Make traintainers have triple the chance to be picked as a reviewer\n  reviewer = (reviewers + traintainers + traintainers).sample(random: rng)\n  maintainer = maintainers.sample(random: rng)\n\n  [reviewer.username, maintainer.username]\nend\n\ndef random_branch_name\n  FFaker::Filesystem.file_name\nend\n\nFFaker::Random.seed = 123\nteam = Foo.new.project_team\nresults = Hash.new(0)\n\n10_000.times do\n  reviewer, maintainer = spin(team, 'gitlab-ce', 'backend', random_branch_name)\n\n  results[reviewer] += 1\n  results[maintainer] += 1\nend\n\nresults.sort_by(&:last).reverse.each do |username, picked|\n  puts \"#{username}: #{picked}\"\nend; nil\n```\n\nThis should output the same for you as it does for me, because we seed the branch names too!\n\n```\ndzaporozhets: 799\nmkozono: 797\ngrzesiek: 794\ngodfat: 793\nDouweM: 788\nnick.thomas: 773\ntkuah: 764\nstanhu: 761\nayufan: 759\njprovaznik: 758\njameslopez: 757\ndbalexandre: 754\nsmcgivern: 751\nsplattael: 744\nashmckenzie: 741\nrspeicher: 739\njarka: 738\nrymai: 735\nreprazent: 729\nmayra-cabrera: 713\nmdelaossa: 273\nrdavila: 269\nvsizov: 260\ntheoretick: 257\noswaldo: 255\nifarkas: 255\nbrytannia: 248\nengwan: 248\nfelipe_artur: 247\nrpereira2: 243\ntoon: 239\nrossfuhrman: 236\njamedjo: 235\nfjsanpedro: 229\nmatteeyah: 226\nbrodock: 226\nvzagorodny: 222\nzj: 220\nmarkglenfletcher: 219\ndosuken123: 206\n```\n\nCloses https://gitlab.com/gitlab-org/gitlab-ce/issues/57766.",
       "state": "merged",
       "created_at": "2019-04-08T10:59:38.140Z",
       "updated_at": "2019-05-02T14:34:54.068Z",

source/platforms/gitlab/_tests/fixtures/getMergeRequestInfo.json

@@ -10,7 +10,7 @@
       "iid": 27117,
       "project_id": 13083,
       "title": "Stable reviewer roulette",
-      "description": "Change reviewer roulette to always pick the same reviewers for the same\nbranch name. We do this by:\n\n1. Making the branch name 'canonical' across CE and EE by stripping a\n   leading 'ce-' or 'ee-' and a trailing '-ce' or '-ee'. If people are\n   following our branch naming guidelines, this should give the same\n   branch name in both repos.\n2. Converting the branch name to a stable integer by taking the integer\n   form of its MD5.\n3. Passing that integer as a seed to Ruby's `Random` class, which 'may\n   be used to ensure repeatable sequences of pseudo-random numbers\n   between different runs of the program' (from the Ruby documentation).\n\nThe upshot is that the same branch name (in CE and EE) should always\npick the same reviewers, and those should be evenly distributed across\nthe set of possible reviewers due to the use of MD5.\n\nAgain, I have a test script:\n\n```ruby\nrequire 'ffaker'\n\nclass Foo\n  include Gitlab::Danger::Helper\nend\n\ndef spin(team, project, category, branch_name)\n  # Strip leading and trailing CE/EE markers\n  canonical_branch_name = branch_name.gsub(/^[ce]e-/, '').gsub(/-[ce]e$/, '')\n  rng = Random.new(Digest::MD5.hexdigest(canonical_branch_name).to_i(16))\n\n  reviewers = team.select { |member| member.reviewer?(project, category) }\n  traintainers = team.select { |member| member.traintainer?(project, category) }\n  maintainers = team.select { |member| member.maintainer?(project, category) }\n\n  # TODO: filter out people who are currently not in the office\n  # https://gitlab.com/gitlab-org/gitlab-ce/issues/57652\n  #\n  # TODO: take CODEOWNERS into account?\n  # https://gitlab.com/gitlab-org/gitlab-ce/issues/57653\n\n  # Make traintainers have triple the chance to be picked as a reviewer\n  reviewer = (reviewers + traintainers + traintainers).sample(random: rng)\n  maintainer = maintainers.sample(random: rng)\n\n  [reviewer.username, maintainer.username]\nend\n\ndef random_branch_name\n  FFaker::Filesystem.file_name\nend\n\nFFaker::Random.seed = 123\nteam = Foo.new.project_team\nresults = Hash.new(0)\n\n10_000.times do\n  reviewer, maintainer = spin(team, 'gitlab-foss', 'backend', random_branch_name)\n\n  results[reviewer] += 1\n  results[maintainer] += 1\nend\n\nresults.sort_by(&:last).reverse.each do |username, picked|\n  puts \"#{username}: #{picked}\"\nend; nil\n```\n\nThis should output the same for you as it does for me, because we seed the branch names too!\n\n```\ndzaporozhets: 799\nmkozono: 797\ngrzesiek: 794\ngodfat: 793\nDouweM: 788\nnick.thomas: 773\ntkuah: 764\nstanhu: 761\nayufan: 759\njprovaznik: 758\njameslopez: 757\ndbalexandre: 754\nsmcgivern: 751\nsplattael: 744\nashmckenzie: 741\nrspeicher: 739\njarka: 738\nrymai: 735\nreprazent: 729\nmayra-cabrera: 713\nmdelaossa: 273\nrdavila: 269\nvsizov: 260\ntheoretick: 257\noswaldo: 255\nifarkas: 255\nbrytannia: 248\nengwan: 248\nfelipe_artur: 247\nrpereira2: 243\ntoon: 239\nrossfuhrman: 236\njamedjo: 235\nfjsanpedro: 229\nmatteeyah: 226\nbrodock: 226\nvzagorodny: 222\nzj: 220\nmarkglenfletcher: 219\ndosuken123: 206\n```\n\nCloses https://gitlab.com/gitlab-org/gitlab-ce/issues/57766.",
+      "description": "Change reviewer roulette to always pick the same reviewers for the same\nbranch name. We do this by:\n\n1. Making the branch name 'canonical' across CE and EE by stripping a\n   leading 'ce-' or 'ee-' and a trailing '-ce' or '-ee'. If people are\n   following our branch naming guidelines, this should give the same\n   branch name in both repos.\n2. Converting the branch name to a stable integer by taking the integer\n   form of its MD5.\n3. Passing that integer as a seed to Ruby's `Random` class, which 'may\n   be used to ensure repeatable sequences of pseudo-random numbers\n   between different runs of the program' (from the Ruby documentation).\n\nThe upshot is that the same branch name (in CE and EE) should always\npick the same reviewers, and those should be evenly distributed across\nthe set of possible reviewers due to the use of MD5.\n\nAgain, I have a test script:\n\n```ruby\nrequire 'ffaker'\n\nclass Foo\n  include Gitlab::Danger::Helper\nend\n\ndef spin(team, project, category, branch_name)\n  # Strip leading and trailing CE/EE markers\n  canonical_branch_name = branch_name.gsub(/^[ce]e-/, '').gsub(/-[ce]e$/, '')\n  rng = Random.new(Digest::MD5.hexdigest(canonical_branch_name).to_i(16))\n\n  reviewers = team.select { |member| member.reviewer?(project, category) }\n  traintainers = team.select { |member| member.traintainer?(project, category) }\n  maintainers = team.select { |member| member.maintainer?(project, category) }\n\n  # TODO: filter out people who are currently not in the office\n  # https://gitlab.com/gitlab-org/gitlab-ce/issues/57652\n  #\n  # TODO: take CODEOWNERS into account?\n  # https://gitlab.com/gitlab-org/gitlab-ce/issues/57653\n\n  # Make traintainers have triple the chance to be picked as a reviewer\n  reviewer = (reviewers + traintainers + traintainers).sample(random: rng)\n  maintainer = maintainers.sample(random: rng)\n\n  [reviewer.username, maintainer.username]\nend\n\ndef random_branch_name\n  FFaker::Filesystem.file_name\nend\n\nFFaker::Random.seed = 123\nteam = Foo.new.project_team\nresults = Hash.new(0)\n\n10_000.times do\n  reviewer, maintainer = spin(team, 'gitlab-ce', 'backend', random_branch_name)\n\n  results[reviewer] += 1\n  results[maintainer] += 1\nend\n\nresults.sort_by(&:last).reverse.each do |username, picked|\n  puts \"#{username}: #{picked}\"\nend; nil\n```\n\nThis should output the same for you as it does for me, because we seed the branch names too!\n\n```\ndzaporozhets: 799\nmkozono: 797\ngrzesiek: 794\ngodfat: 793\nDouweM: 788\nnick.thomas: 773\ntkuah: 764\nstanhu: 761\nayufan: 759\njprovaznik: 758\njameslopez: 757\ndbalexandre: 754\nsmcgivern: 751\nsplattael: 744\nashmckenzie: 741\nrspeicher: 739\njarka: 738\nrymai: 735\nreprazent: 729\nmayra-cabrera: 713\nmdelaossa: 273\nrdavila: 269\nvsizov: 260\ntheoretick: 257\noswaldo: 255\nifarkas: 255\nbrytannia: 248\nengwan: 248\nfelipe_artur: 247\nrpereira2: 243\ntoon: 239\nrossfuhrman: 236\njamedjo: 235\nfjsanpedro: 229\nmatteeyah: 226\nbrodock: 226\nvzagorodny: 222\nzj: 220\nmarkglenfletcher: 219\ndosuken123: 206\n```\n\nCloses https://gitlab.com/gitlab-org/gitlab-ce/issues/57766.",
       "state": "merged",
       "created_at": "2019-04-08T10:59:38.140Z",
       "updated_at": "2019-05-02T14:34:54.068Z",