)]}'
{"taskflow/types/graph.py":[{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"47af906e4d1a4e9c6c6d90974cb96c45f2ad5eb0","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        \"\"\"Pretty formats your graph into a string.\"\"\""},{"line_number":65,"context_line":"        return os.linesep.join(_common_format(self, \"\u003c-\u003e\"))"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def fresh_copy(self):"},{"line_number":68,"context_line":"        \"\"\"Return a fresh copy graph with the same data structure."},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        A fresh copy has no nodes, edges or graph attributes. It is"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_0fb5f4ec","line":67,"updated":"2018-06-25 16:35:56.000000000","message":"I don\u0027t understand the purpose of these methods.  They don\u0027t seem to be used in this patch (at least glancing through it) and there doesn\u0027t seem to be any benefit over just calling the constructor of the class since that\u0027s all this does.","commit_id":"c294097cce93244c0a4474c16f5053294dcd261d"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4329e76d340dc3dec52f3e53d65a3f5559602802","unresolved":false,"context_lines":[{"line_number":20,"context_line":"import networkx as nx"},{"line_number":21,"context_line":"from networkx.drawing import nx_pydot"},{"line_number":22,"context_line":"import six"},{"line_number":23,"context_line":"from taskflow.utils.misc import nx_version"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def _common_format(g, edge_notation):"}],"source_content_type":"text/x-python","patch_set":15,"id":"5f7c97a3_bc88dcc2","line":23,"updated":"2018-07-05 21:36:57.000000000","message":"Nit, but local imports should be in their own grouping after third party imports, so this should have a blank line after the six import.","commit_id":"172502e8a17b79b39e0b7f58e97c631c4c2cb42d"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"24bc789245b8c9e10d7cca1955195e87bf7dd5e4","unresolved":false,"context_lines":[{"line_number":20,"context_line":"import networkx as nx"},{"line_number":21,"context_line":"from networkx.drawing import nx_pydot"},{"line_number":22,"context_line":"import six"},{"line_number":23,"context_line":"from taskflow.utils.misc import nx_version"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"def _common_format(g, edge_notation):"}],"source_content_type":"text/x-python","patch_set":15,"id":"5f7c97a3_f7598f5a","line":23,"in_reply_to":"5f7c97a3_bc88dcc2","updated":"2018-07-05 23:01:18.000000000","message":"Added blank line.","commit_id":"172502e8a17b79b39e0b7f58e97c631c4c2cb42d"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"62f714018a901043dfd11c7be15e6de1e836c232","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from networkx.drawing import nx_pydot"},{"line_number":22,"context_line":"import six"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from taskflow.utils.misc import nx_version"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"def _common_format(g, edge_notation):"}],"source_content_type":"text/x-python","patch_set":16,"id":"5f7c97a3_b56420b5","line":24,"updated":"2018-07-10 20:56:17.000000000","message":"Nit: Our style rules say to only import modules, and it looks like this is importing a function.","commit_id":"145b97adc55bef8b9728fde34b806b58b6fce0c2"}],"test-requirements.txt":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4329e76d340dc3dec52f3e53d65a3f5559602802","unresolved":false,"context_lines":[{"line_number":29,"context_line":"mock\u003e\u003d2.0.0 # BSD"},{"line_number":30,"context_line":"testtools\u003e\u003d2.2.0 # MIT"},{"line_number":31,"context_line":"testscenarios\u003e\u003d0.4 # Apache-2.0/BSD"},{"line_number":32,"context_line":"testrepository\u003e\u003d0.0.18 # Apache-2.0/BSD"},{"line_number":33,"context_line":"doc8\u003e\u003d0.6.0 # Apache-2.0"},{"line_number":34,"context_line":"sphinx!\u003d1.6.6,!\u003d1.6.7,\u003e\u003d1.6.2 # BSD"}],"source_content_type":"text/plain","patch_set":15,"id":"5f7c97a3_7cc004e1","line":32,"updated":"2018-07-05 21:36:57.000000000","message":"Can you explain why this is being added?","commit_id":"172502e8a17b79b39e0b7f58e97c631c4c2cb42d"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"24bc789245b8c9e10d7cca1955195e87bf7dd5e4","unresolved":false,"context_lines":[{"line_number":29,"context_line":"mock\u003e\u003d2.0.0 # BSD"},{"line_number":30,"context_line":"testtools\u003e\u003d2.2.0 # MIT"},{"line_number":31,"context_line":"testscenarios\u003e\u003d0.4 # Apache-2.0/BSD"},{"line_number":32,"context_line":"testrepository\u003e\u003d0.0.18 # Apache-2.0/BSD"},{"line_number":33,"context_line":"doc8\u003e\u003d0.6.0 # Apache-2.0"},{"line_number":34,"context_line":"sphinx!\u003d1.6.6,!\u003d1.6.7,\u003e\u003d1.6.2 # BSD"}],"source_content_type":"text/plain","patch_set":15,"id":"5f7c97a3_f782efa4","line":32,"in_reply_to":"5f7c97a3_7cc004e1","updated":"2018-07-05 23:01:18.000000000","message":"I think that requirements test were failing when it was not added, if i remember , testr command was missing and because of this test failed .. So, I have added  this to test-requirements .. Also , other openstack projects has this entry in test-requirements.txt .. So, I\u0027ve add it to taskflow as well.","commit_id":"172502e8a17b79b39e0b7f58e97c631c4c2cb42d"}]}