)]}'
{"pbr/tests/testpackage/test-requirements.txt":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ddc6f0369b24b641df28a69f8549feb4d18f04c9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"ordereddict;python_version\u003d\u003d\u00272.6\u0027"},{"line_number":2,"context_line":"requests-mock"}],"source_content_type":"text/plain","patch_set":1,"id":"3f4b6375_d928b598","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":13},"updated":"2017-11-09 03:43:17.000000000","message":"This appears to be here merely here for tests. We should note this in the comments.\n\nAlso, I wonder do we check this as part of the requirements job? If not, we probably should","commit_id":"53fd803372db2fdaa55d08e89202420fcca24a20"}],"pbr/util.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"aeaf94e5bb85562367e64056e704519d658d8978","unresolved":false,"context_lines":[{"line_number":404,"context_line":"        extras \u003d config[\u0027extras\u0027]"},{"line_number":405,"context_line":"        # Add contents of test-requirements, if any, into an extra named"},{"line_number":406,"context_line":"        # \u0027test\u0027 if one does not already exist."},{"line_number":407,"context_line":"        if \u0027test\u0027 not in extras:"},{"line_number":408,"context_line":"            from pbr import packaging"},{"line_number":409,"context_line":"            extras[\u0027test\u0027] \u003d \"\\n\".join(packaging.parse_requirements("},{"line_number":410,"context_line":"                packaging.TEST_REQUIREMENTS_FILES)).replace(\u0027;\u0027, \u0027:\u0027)"},{"line_number":411,"context_line":""},{"line_number":412,"context_line":"        for extra in extras:"},{"line_number":413,"context_line":"            extra_requirements \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_8e2a5129","line":410,"range":{"start_line":407,"start_character":0,"end_line":410,"end_character":69},"updated":"2017-09-28 12:58:37.000000000","message":"Actually, what happens when this file isn\u0027t present? Does \u0027packaging.parse_requirements\u0027 handle non-existent files as expected? I\u0027d really like a test to validate behavior for this case","commit_id":"53fd803372db2fdaa55d08e89202420fcca24a20"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ddc6f0369b24b641df28a69f8549feb4d18f04c9","unresolved":false,"context_lines":[{"line_number":404,"context_line":"        extras \u003d config[\u0027extras\u0027]"},{"line_number":405,"context_line":"        # Add contents of test-requirements, if any, into an extra named"},{"line_number":406,"context_line":"        # \u0027test\u0027 if one does not already exist."},{"line_number":407,"context_line":"        if \u0027test\u0027 not in extras:"},{"line_number":408,"context_line":"            from pbr import packaging"},{"line_number":409,"context_line":"            extras[\u0027test\u0027] \u003d \"\\n\".join(packaging.parse_requirements("},{"line_number":410,"context_line":"                packaging.TEST_REQUIREMENTS_FILES)).replace(\u0027;\u0027, \u0027:\u0027)"},{"line_number":411,"context_line":""},{"line_number":412,"context_line":"        for extra in extras:"},{"line_number":413,"context_line":"            extra_requirements \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_195acd4f","line":410,"range":{"start_line":407,"start_character":0,"end_line":410,"end_character":69},"in_reply_to":"7f515b1d_8e2a5129","updated":"2017-11-09 03:43:17.000000000","message":"Yes, it does, and we own packaging so there\u0027s not _really_ any need to worry about this here. I\u0027m going to add a test to keep myself happy though","commit_id":"53fd803372db2fdaa55d08e89202420fcca24a20"}]}