)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"5bf278cea480f943a924f98cc6de4b6773ade5e2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Adam Richter \u003cadamr@checkpoint.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-02-11 15:42:56 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Linkify BuildOutput"},{"line_number":8,"context_line":"Show URLs in message as links"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I6ed5b1331ab566571861fcff1b44b9748d13d2a5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ec0963f8_d7447877","line":7,"updated":"2021-02-11 19:35:21.000000000","message":"Please insert a blank line here between the commit subject and body.","commit_id":"6c52bd19d6c3ce61d308a6aea5318fb67e53919c"},{"author":{"_account_id":32208,"name":"Ron Izraeli","email":"roniz@checkpoint.com"},"change_message_id":"46cd69947c1bda3818992268f1f79fe7fefca4b5","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Adam Richter \u003cadamr@checkpoint.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-02-11 15:42:56 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Linkify BuildOutput"},{"line_number":8,"context_line":"Show URLs in message as links"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I6ed5b1331ab566571861fcff1b44b9748d13d2a5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"77f037ae_985e8fff","line":7,"in_reply_to":"ec0963f8_d7447877","updated":"2021-02-12 08:50:37.000000000","message":"Fixed","commit_id":"6c52bd19d6c3ce61d308a6aea5318fb67e53919c"}],"web/package.json":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"3f45d4edf8cd88530a85ed6da5f5d2b5178f1b0e","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \"react-router-dom\": \"^4.3.1\","},{"line_number":28,"context_line":"    \"react-scripts\": \"3.4.1\","},{"line_number":29,"context_line":"    \"react-select\": \"3.1.0\","},{"line_number":30,"context_line":"    \"react-linkify\": \"^1.0.0-alpha\","},{"line_number":31,"context_line":"    \"redux\": \"^4.0.5\","},{"line_number":32,"context_line":"    \"redux-immutable-state-invariant\": \"^2.1.0\","},{"line_number":33,"context_line":"    \"redux-thunk\": \"^2.3.0\","}],"source_content_type":"application/json","patch_set":4,"id":"5b042a3e_9d26e773","line":30,"updated":"2021-02-12 16:20:11.000000000","message":"No change needed from my perspective, I was just letting other reviewers know the license state for the proposed additional dependency so they wouldn\u0027t need to redo the same research themselves.","commit_id":"6c52bd19d6c3ce61d308a6aea5318fb67e53919c"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"5bf278cea480f943a924f98cc6de4b6773ade5e2","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \"react-router-dom\": \"^4.3.1\","},{"line_number":28,"context_line":"    \"react-scripts\": \"3.4.1\","},{"line_number":29,"context_line":"    \"react-select\": \"3.1.0\","},{"line_number":30,"context_line":"    \"react-linkify\": \"^1.0.0-alpha\","},{"line_number":31,"context_line":"    \"redux\": \"^4.0.5\","},{"line_number":32,"context_line":"    \"redux-immutable-state-invariant\": \"^2.1.0\","},{"line_number":33,"context_line":"    \"redux-thunk\": \"^2.3.0\","}],"source_content_type":"application/json","patch_set":4,"id":"b2db835a_b9c82ec9","line":30,"updated":"2021-02-11 19:35:21.000000000","message":"This asserts it\u0027s distributed under an MIT (Expat) license, which is Apache compatible, so should be fine as far as legality is concerned.","commit_id":"6c52bd19d6c3ce61d308a6aea5318fb67e53919c"},{"author":{"_account_id":32208,"name":"Ron Izraeli","email":"roniz@checkpoint.com"},"change_message_id":"2665fc2f37a4ec17b412e3ec1c968a3ee1ba11ed","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \"react-router-dom\": \"^4.3.1\","},{"line_number":28,"context_line":"    \"react-scripts\": \"3.4.1\","},{"line_number":29,"context_line":"    \"react-select\": \"3.1.0\","},{"line_number":30,"context_line":"    \"react-linkify\": \"^1.0.0-alpha\","},{"line_number":31,"context_line":"    \"redux\": \"^4.0.5\","},{"line_number":32,"context_line":"    \"redux-immutable-state-invariant\": \"^2.1.0\","},{"line_number":33,"context_line":"    \"redux-thunk\": \"^2.3.0\","}],"source_content_type":"application/json","patch_set":4,"id":"13d043c1_3477e95c","line":30,"in_reply_to":"366c7be3_f6ef76d6","updated":"2021-02-12 16:31:38.000000000","message":"Done","commit_id":"6c52bd19d6c3ce61d308a6aea5318fb67e53919c"},{"author":{"_account_id":32208,"name":"Ron Izraeli","email":"roniz@checkpoint.com"},"change_message_id":"46cd69947c1bda3818992268f1f79fe7fefca4b5","unresolved":true,"context_lines":[{"line_number":27,"context_line":"    \"react-router-dom\": \"^4.3.1\","},{"line_number":28,"context_line":"    \"react-scripts\": \"3.4.1\","},{"line_number":29,"context_line":"    \"react-select\": \"3.1.0\","},{"line_number":30,"context_line":"    \"react-linkify\": \"^1.0.0-alpha\","},{"line_number":31,"context_line":"    \"redux\": \"^4.0.5\","},{"line_number":32,"context_line":"    \"redux-immutable-state-invariant\": \"^2.1.0\","},{"line_number":33,"context_line":"    \"redux-thunk\": \"^2.3.0\","}],"source_content_type":"application/json","patch_set":4,"id":"366c7be3_f6ef76d6","line":30,"in_reply_to":"b2db835a_b9c82ec9","updated":"2021-02-12 08:50:37.000000000","message":"Any change required?","commit_id":"6c52bd19d6c3ce61d308a6aea5318fb67e53919c"}],"web/src/containers/build/BuildOutput.jsx":[{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"cfe55bb7cf6e7291a55ca71ccac5f8e8b1c97784","unresolved":true,"context_lines":[{"line_number":139,"context_line":"    const linkify_options \u003d {"},{"line_number":140,"context_line":"      validate: {"},{"line_number":141,"context_line":"        url: function (value) {"},{"line_number":142,"context_line":"          return /^.+:\\/\\//.test(value)"},{"line_number":143,"context_line":"        }}}"},{"line_number":144,"context_line":"    return ("},{"line_number":145,"context_line":"      \u003cLinkify options\u003d{linkify_options}\u003e"}],"source_content_type":"text/jsx","patch_set":6,"id":"c89457fc_e0f47432","line":142,"range":{"start_line":142,"start_character":17,"end_line":142,"end_character":27},"updated":"2021-02-12 22:42:02.000000000","message":"That doesn\u0027t seem to match links that are prefixed with some text, e.g. `Get:1 http://mirro...`, perhaps the `^` could be dropped?","commit_id":"945b46c311be9d7fd8873c68f16ef687f582962f"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"f200512aeced634c903e76eecfc3a8eb8b763a0f","unresolved":false,"context_lines":[{"line_number":139,"context_line":"    const linkify_options \u003d {"},{"line_number":140,"context_line":"      validate: {"},{"line_number":141,"context_line":"        url: function (value) {"},{"line_number":142,"context_line":"          return /^.+:\\/\\//.test(value)"},{"line_number":143,"context_line":"        }}}"},{"line_number":144,"context_line":"    return ("},{"line_number":145,"context_line":"      \u003cLinkify options\u003d{linkify_options}\u003e"}],"source_content_type":"text/jsx","patch_set":6,"id":"16aa5b2c_919090f8","line":142,"updated":"2021-02-13 15:55:06.000000000","message":"This regex looks like it might make the algo O(n^2) if it\u0027s applied to every point in the string.","commit_id":"945b46c311be9d7fd8873c68f16ef687f582962f"},{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"6339bcf203d6680b11190ee1713bd10659d7468b","unresolved":true,"context_lines":[{"line_number":139,"context_line":"    const linkify_options \u003d {"},{"line_number":140,"context_line":"      validate: {"},{"line_number":141,"context_line":"        url: function (value) {"},{"line_number":142,"context_line":"          return /^.+:\\/\\//.test(value)"},{"line_number":143,"context_line":"        }}}"},{"line_number":144,"context_line":"    return ("},{"line_number":145,"context_line":"      \u003cLinkify options\u003d{linkify_options}\u003e"}],"source_content_type":"text/jsx","patch_set":6,"id":"4c2f35bd_8b5b68b6","line":142,"range":{"start_line":142,"start_character":17,"end_line":142,"end_character":27},"in_reply_to":"71238175_e61b2cda","updated":"2021-02-12 22:59:02.000000000","message":"Oops my bad, I though this applied to the console tab too, this is working as expected.","commit_id":"945b46c311be9d7fd8873c68f16ef687f582962f"},{"author":{"_account_id":32209,"name":"Adam Richter","email":"adamr@checkpoint.com"},"change_message_id":"b298118370b71dba35a5fb7224fb67487af72644","unresolved":true,"context_lines":[{"line_number":139,"context_line":"    const linkify_options \u003d {"},{"line_number":140,"context_line":"      validate: {"},{"line_number":141,"context_line":"        url: function (value) {"},{"line_number":142,"context_line":"          return /^.+:\\/\\//.test(value)"},{"line_number":143,"context_line":"        }}}"},{"line_number":144,"context_line":"    return ("},{"line_number":145,"context_line":"      \u003cLinkify options\u003d{linkify_options}\u003e"}],"source_content_type":"text/jsx","patch_set":6,"id":"71238175_e61b2cda","line":142,"range":{"start_line":142,"start_character":17,"end_line":142,"end_character":27},"in_reply_to":"c89457fc_e0f47432","updated":"2021-02-12 22:51:14.000000000","message":"I think we only want to make sure there is a \"\u003cprotocol\u003e://\" at the beginning of the string matched by Linkify, hence the \u0027^\u0027.\n\nNot sure I understood your example, what\u0027s the full text with the link that should be matched?","commit_id":"945b46c311be9d7fd8873c68f16ef687f582962f"}]}