)]}'
{"playbooks/library/git_requirements.py":[{"author":{"_account_id":29865,"name":"Georgina Shippey","email":"georgina.shippey@outlook.com","username":"gshippey"},"change_message_id":"6a1a8459e8fa5ce91177543bee5d8dd31af1c34c","unresolved":true,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"def check_out_version(repo, version, pull\u003dFalse, force\u003dFalse,"},{"line_number":88,"context_line":"                      refspec\u003dNone, tag\u003dFalse, depth\u003d10):"},{"line_number":89,"context_line":"    try:"},{"line_number":90,"context_line":"        repo.git.fetch(force\u003dforce, refspec\u003drefspec, depth\u003ddepth)"},{"line_number":91,"context_line":"    except Exception as e:"},{"line_number":92,"context_line":"        return [\"Failed to fetch %s\\n%s\" % (repo.working_dir, str(e))]"}],"source_content_type":"text/x-python","patch_set":1,"id":"08040ebf_4eb796d6","line":89,"updated":"2021-05-12 15:38:13.000000000","message":"Unnecessarily fetching twice when tags\u003dTrue.\n\nShould just be able to do:\n\nif not tag:\n    normal fetch\nif tag:\n    fetch --tags\n\nFrom git docs: \n--tags Fetch all tags from the remote (i.e., fetch remote tags refs/tags/* into local tags with the same name), in addition to whatever else would otherwise be fetched.\n\nMost important part is \u0027in addition\u0027 to the normal stuff fetched.","commit_id":"cc3d29f67ad73b5666abf0f8feddc70543cd780a"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"ad2e4330e80af3be9b898ef3932970e7d2af58e0","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"def check_out_version(repo, version, pull\u003dFalse, force\u003dFalse,"},{"line_number":88,"context_line":"                      refspec\u003dNone, tag\u003dFalse, depth\u003d10):"},{"line_number":89,"context_line":"    try:"},{"line_number":90,"context_line":"        repo.git.fetch(force\u003dforce, refspec\u003drefspec, depth\u003ddepth)"},{"line_number":91,"context_line":"    except Exception as e:"},{"line_number":92,"context_line":"        return [\"Failed to fetch %s\\n%s\" % (repo.working_dir, str(e))]"}],"source_content_type":"text/x-python","patch_set":1,"id":"0718c5be_27e6e695","line":89,"in_reply_to":"08040ebf_4eb796d6","updated":"2021-06-01 12:27:19.000000000","message":"Sorry, just noticed your comment :( Fixed and just removed this if tag condition overall","commit_id":"cc3d29f67ad73b5666abf0f8feddc70543cd780a"}]}