)]}'
{"web/src/containers/build/Console.jsx":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"54daf6f564fb7f216525d221ae088c5db3a3ad50","unresolved":false,"context_lines":[{"line_number":211,"context_line":""},{"line_number":212,"context_line":"    hostTaskStats(this.state, host)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    if (makeTaskPath(taskPath) \u003d\u003d\u003d makeTaskPath(displayPath.slice(0, taskPath.length))) {"},{"line_number":215,"context_line":"      this.state.showModal \u003d true"},{"line_number":216,"context_line":"    }"},{"line_number":217,"context_line":"  }"}],"source_content_type":"text/jsx","patch_set":2,"id":"7faddb67_4474e6fe","line":214,"range":{"start_line":214,"start_character":69,"end_line":214,"end_character":84},"updated":"2019-08-08 19:21:17.000000000","message":"Could this result in collisions where displaypath[0:len(taskPath)] produces a displaypath substring that matches makeTaskPath(taskPath) but the longer complete displayPath string will not?","commit_id":"1b24990387061981c669a4662aa9d101046c39ec"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"db2fb93c63611a415a00fe9dabacc5d6321c032b","unresolved":false,"context_lines":[{"line_number":211,"context_line":""},{"line_number":212,"context_line":"    hostTaskStats(this.state, host)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    if (makeTaskPath(taskPath) \u003d\u003d\u003d makeTaskPath(displayPath.slice(0, taskPath.length))) {"},{"line_number":215,"context_line":"      this.state.showModal \u003d true"},{"line_number":216,"context_line":"    }"},{"line_number":217,"context_line":"  }"}],"source_content_type":"text/jsx","patch_set":2,"id":"7faddb67_244f6a93","line":214,"updated":"2019-08-08 19:36:10.000000000","message":"Yes, if there\u0027s a host that is \"ubuntu\" and also \"ubuntu-xenial\".  For that matter, the other checks might be suceptible to needlessly expanding playbook 10 when playbook 1 is selected.  We should rework both of those to actually split the paths on the separator.","commit_id":"1b24990387061981c669a4662aa9d101046c39ec"}]}