)]}'
{"doc/source/_exts/deliverables.py":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"92fd1937024048cbdd0fad1b00a14d20a5b82f54","unresolved":false,"context_lines":[{"line_number":68,"context_line":""},{"line_number":69,"context_line":"def _get_category(deliv):"},{"line_number":70,"context_line":"    model \u003d deliv.model"},{"line_number":71,"context_line":"    if model \u003d\u003d \u0027cycle-trailing\u0027:"},{"line_number":72,"context_line":"        return \u0027cycle-trailing\u0027"},{"line_number":73,"context_line":"    if model \u003d\u003d \u0027abandoned\u0027:"},{"line_number":74,"context_line":"        return \u0027abandoned\u0027"},{"line_number":75,"context_line":"    return deliv.type"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_76a71c52","line":74,"range":{"start_line":71,"start_character":4,"end_line":74,"end_character":26},"updated":"2019-12-20 16:16:37.000000000","message":"These changes LGTM in the current state but this could be:\n\n```\nspecific_models \u003d [\u0027cycle-trailing\u0027, \u0027abandoned\u0027]\nif deliv.model in specific_models:\n    return deliv.model\nreturn deliv.type\n```\n\nto reduce number of tests.\nAnyway it\u0027s just a suggest I\u0027ll +1","commit_id":"2860ee81c92fa4a15f3b1076e362aaf9dc793347"}]}