)]}'
{"pbr/packaging.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3638b7442bd818c2824a38b592458f8befc04097","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"REQUIREMENTS_FILES \u003d (\u0027requirements.txt\u0027, \u0027tools/pip-requires\u0027)"},{"line_number":48,"context_line":"TEST_REQUIREMENTS_FILES \u003d (\u0027test-requirements.txt\u0027, \u0027tools/test-requires\u0027)"},{"line_number":49,"context_line":"DOCS_REQUIREMENTS_FILES \u003d (\u0027doc/requirements.txt\u0027,)"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"def get_requirements_files():"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f485f77_83189e75","line":49,"range":{"start_line":49,"start_character":27,"end_line":49,"end_character":49},"updated":"2017-11-09 06:38:36.000000000","message":"Can you extend this to include \u0027docs/requirements.txt\u0027 since I\u0027ve found this to be more common outside OpenStack","commit_id":"9c6295c2c2265b9a25f2d9b1403252365bdcb94b"}],"pbr/util.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c853b6be07a10d42ffecc165447dc5445161c08e","unresolved":false,"context_lines":[{"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":"        # Add contents of doc/requirements.txt, if any, into an extra named"},{"line_number":412,"context_line":"        # \u0027docs\u0027 if one does not already exist."},{"line_number":413,"context_line":"        if \u0027docs\u0027 not in extras:"},{"line_number":414,"context_line":"            from pbr import packaging"},{"line_number":415,"context_line":"            extras[\u0027docs\u0027] \u003d \"\\n\".join(packaging.parse_requirements("},{"line_number":416,"context_line":"                packaging.DOCS_REQUIREMENTS_FILES)).replace(\u0027;\u0027, \u0027:\u0027)"},{"line_number":417,"context_line":""},{"line_number":418,"context_line":"        for extra in extras:"},{"line_number":419,"context_line":"            extra_requirements \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_2e6b5d85","line":416,"range":{"start_line":411,"start_character":0,"end_line":416,"end_character":69},"updated":"2017-09-28 12:55:47.000000000","message":"Wouldn\u0027t this create a \u0027docs\u0027 attribute even \u0027packaging.DOCS_REQUIREMENTS_FILES\u0027 is empty or does not exist? I wonder if this is correct behavior, or if we should instead either leave it unset or set it to the combination of \u0027test\u0027 and default requirements?","commit_id":"4fb196cd0d15f285a2493af1b10cee14b134b9ba"}]}