{"push_id":1130595195,"size":7,"distinct_size":7,"ref":"refs/heads/master","head":"85495c8c85533e2d4156231fd2535270afffef5a","before":"f2caad2467f318ec1359ee9b03509e831cde9d16","commits":[{"sha":"77a1565c806fece3a1d664c2c069f38dcb371c5c","author":{"email":"ce5845b12f2f1d773f833057124e88241e8fbaf1@gmail.com","name":"Connor Shea"},"message":"Replace `link_to_gfm` with `link_to` in merge request and issue titles.\n\nPreviously if an issue had an emoji for its name it would be unclickable. Similarly, links were rendered inline if the title was something like “Fixes a bug with https://google.com”. This confused some users, so it’s removed.\n\nOn the issue/MR page itself, the formatting is preserved. This only effects index pages.\n\nThis is also ever so slightly more performant :)\n\nThis fixes #17614 and #17230.","distinct":true,"url":"https://api.github.com/repos/gitlabhq/gitlabhq/commits/77a1565c806fece3a1d664c2c069f38dcb371c5c"},{"sha":"6d2a0a6b9b97570b07c3b3aec74c6420b1919403","author":{"email":"9ad03ba8d7b02c87ca30b6d4a2d7ed611a7f1fd2@gmail.com","name":"Josh Frye"},"message":"Remove main_language and tests","distinct":true,"url":"https://api.github.com/repos/gitlabhq/gitlabhq/commits/6d2a0a6b9b97570b07c3b3aec74c6420b1919403"},{"sha":"d5e93deb012423697158576cfd1c22ed582cb23c","author":{"email":"9ad03ba8d7b02c87ca30b6d4a2d7ed611a7f1fd2@gmail.com","name":"Josh Frye"},"message":"Update changelog","distinct":true,"url":"https://api.github.com/repos/gitlabhq/gitlabhq/commits/d5e93deb012423697158576cfd1c22ed582cb23c"},{"sha":"d7bd7a3e7ef2e64f50d5404fe3d90f0c1ff57705","author":{"email":"b1c1d8736f20db3fb6c1c66bb1455ed43909f0d8@iamphill.com","name":"Phil Hughes"},"message":"Fixed JS error when trying to remove discussion form\n\nCloses #17778","distinct":true,"url":"https://api.github.com/repos/gitlabhq/gitlabhq/commits/d7bd7a3e7ef2e64f50d5404fe3d90f0c1ff57705"},{"sha":"44cc765a8c42a38ebfc3caa16d4a19f56b850aff","author":{"email":"c05b58e00d2949f72de3e884ec9d41e9e0eddaec@gitlab.com","name":"Jacob Schatz"},"message":"Merge branch 'link-to-in-issues-index' into 'master'\r\n\r\nReplace `link_to_gfm` with `link_to` in merge request and issue titles.\r\n\r\n## What does this MR do?\r\nThis removed GitLab Markdown Formatting from issue titles due to issues and confusion it caused as well as performance reasons.\r\n\r\nOn the issue/MR page itself, the formatting is preserved. This only effects index pages. See:\r\n\r\n![Screen_Shot_2016-05-24_at_3.31.17_PM](/uploads/8857ed7f3cc6a47ea41ca88134e5d17a/Screen_Shot_2016-05-24_at_3.31.17_PM.png)\r\n\r\nI think the trade-off here is worth it, otherwise we'd just be playing wack-a-mole.\r\n\r\n## Are there points in the code the reviewer needs to double check?\r\nI don't think so.\r\n\r\n## Why was this MR needed?\r\nPreviously if an issue had an emoji for its name it would be unclickable. Similarly, links were rendered inline if the title was something like “Fixes a bug with https://google.com”. This confused a number of users, so it’s removed.\r\n\r\nThis is also ever so slightly more performant :)\r\n\r\n## What are the relevant issue numbers?\r\nThis fixes #17614 and #17230.\r\n\r\n## Screenshots (if relevant)\r\n\r\nBefore:\r\n\r\n![Screen_Shot_2016-05-24_at_3.19.52_PM](/uploads/25313af3d3995a096b7bad7c5a26d071/Screen_Shot_2016-05-24_at_3.19.52_PM.png)\r\n\r\nAfter:\r\n\r\n![Screen_Shot_2016-05-24_at_3.20.13_PM](/uploads/f14f09918001bcbac2cd939b016479b2/Screen_Shot_2016-05-24_at_3.20.13_PM.png)\r\n\r\nBefore:\r\n\r\n![Screen_Shot_2016-05-24_at_3.20.48_PM](/uploads/c183ae2c491650c328eb5768f6a55fd9/Screen_Shot_2016-05-24_at_3.20.48_PM.png)\r\n\r\nAfter:\r\n\r\n![Screen_Shot_2016-05-24_at_3.20.42_PM](/uploads/d3431a691768bae2c5244b9b632061b9/Screen_Shot_2016-05-24_at_3.20.42_PM.png)\r\n\r\ncc: @rspeicher @jschatz1 \r\n\r\nSee merge request !4284","distinct":true,"url":"https://api.github.com/repos/gitlabhq/gitlabhq/commits/44cc765a8c42a38ebfc3caa16d4a19f56b850aff"},{"sha":"24b24aff754e3c7fd9ba8306fc3a452338872276","author":{"email":"c05b58e00d2949f72de3e884ec9d41e9e0eddaec@gitlab.com","name":"Jacob Schatz"},"message":"Merge branch 'discussion-outdated-form' into 'master'\r\n\r\nFixed JS error when trying to remove discussion form\r\n\r\n## What does this MR do?\r\n\r\nFixes a JS error which was caused by an ID of the form not matching what was returned by the JSON. Instead of checking that, it gets the current form from the ajax success event.\r\n\r\nThis would only happen on outdated discussions because the ID of the discussion form ends with `-false` because it isn't active. However, the note is added to an active discussion so the ID returned actually ends in `-true` & therefore the JS couldn't find the correct form.\r\n\r\n## What are the relevant issue numbers?\r\n\r\nCloses #17778\r\n\r\nSee merge request !4303","distinct":true,"url":"https://api.github.com/repos/gitlabhq/gitlabhq/commits/24b24aff754e3c7fd9ba8306fc3a452338872276"},{"sha":"85495c8c85533e2d4156231fd2535270afffef5a","author":{"email":"9ad03ba8d7b02c87ca30b6d4a2d7ed611a7f1fd2@gmail.com","name":"Josh Frye"},"message":"Merge branch 'remove-main-language' into 'master'\r\n\r\nRemove main_language and tests. Closes #14937\r\n\r\n\r\n\r\nSee merge request !4293","distinct":true,"url":"https://api.github.com/repos/gitlabhq/gitlabhq/commits/85495c8c85533e2d4156231fd2535270afffef5a"}]}
{"actor":{"display_login":"randx"}}