)]}'
{"doc/source/commands.rst":[{"author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"change_message_id":"487343b1562bf4c0edde662e1f3c22d18b7483d6","unresolved":false,"context_lines":[{"line_number":17,"context_line":"Command Arguments"},{"line_number":18,"context_line":"-----------------"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"  * All long option names use two dashes (\u0027--\u0027) as the prefix and a single dash "},{"line_number":21,"context_line":"    (\u0027-\u0027) as the interpolation character.  Some common options also have the"},{"line_number":22,"context_line":"    traditional single letter name prefixed by a single dash (\u0027-\u0027)."},{"line_number":23,"context_line":"  * Global options generally have a corresponding environment variable that"}],"source_content_type":"text/x-rst","patch_set":2,"id":"AAAAM3%2F%2FLRE%3D","line":20,"updated":"2013-02-04 19:46:12.000000000","message":"Extra space here.","commit_id":"70c15bef268681aad1031faca94508a9aaa27e0d"}],"openstackclient/common/commandmanager.py":[{"author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"change_message_id":"487343b1562bf4c0edde662e1f3c22d18b7483d6","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":"#"},{"line_number":16,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"\"\"\"Modify Cliff\u0027s CommandManager\"\"\""},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FLQc%3D","line":16,"updated":"2013-02-04 19:46:12.000000000","message":"Another minor nit, but would you be able to replace this header with the new one?","commit_id":"70c15bef268681aad1031faca94508a9aaa27e0d"}],"openstackclient/identity/v3/user.py":[{"author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"change_message_id":"487343b1562bf4c0edde662e1f3c22d18b7483d6","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":"#"},{"line_number":16,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"\"\"\"Identity v3 User action implementations\"\"\""},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FLQY%3D","line":16,"updated":"2013-02-04 19:46:12.000000000","message":"And this one too.","commit_id":"70c15bef268681aad1031faca94508a9aaa27e0d"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"e148e1880a166172682de5028e60768554b2acdb","unresolved":false,"context_lines":[{"line_number":139,"context_line":"    def take_action(self, parsed_args):"},{"line_number":140,"context_line":"        self.log.debug(\u0027take_action(%s)\u0027 % parsed_args)"},{"line_number":141,"context_line":"        if parsed_args.long:"},{"line_number":142,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027, \u0027ProjectId\u0027, \u0027Email\u0027, \u0027Enabled\u0027)"},{"line_number":143,"context_line":"        else:"},{"line_number":144,"context_line":"            columns \u003d (\u0027ID\u0027, \u0027Name\u0027)"},{"line_number":145,"context_line":"        data \u003d self.app.client_manager.identity.users.list()"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FLEA%3D","line":142,"updated":"2013-02-04 21:35:15.000000000","message":"Project ID instead of ProjectId?","commit_id":"70c15bef268681aad1031faca94508a9aaa27e0d"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"5b51860394471b80e1c0abc9f0c0a3ea83e016fe","unresolved":false,"context_lines":[{"line_number":65,"context_line":"            dest\u003d\u0027enabled\u0027,"},{"line_number":66,"context_line":"            action\u003d\u0027store_false\u0027,"},{"line_number":67,"context_line":"            help\u003d\u0027Disable user\u0027,"},{"line_number":68,"context_line":"        )"},{"line_number":69,"context_line":"        return parser"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FHho%3D","line":68,"updated":"2013-02-06 17:48:33.000000000","message":"We may want to start removing the --enable option from the create. Since it is enabled by default. Reference: https://review.openstack.org/#/c/20854/4/openstackclient/identity/v3/domain.py\n\nThat being said, we can always change all of these at once, in a different patch.","commit_id":"fbc412e533bd7cb07c6d930e194f660e14b2319f"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"7afd840e418a8e84fed8942c78fa15a3c7d11455","unresolved":false,"context_lines":[{"line_number":65,"context_line":"            dest\u003d\u0027enabled\u0027,"},{"line_number":66,"context_line":"            action\u003d\u0027store_false\u0027,"},{"line_number":67,"context_line":"            help\u003d\u0027Disable user\u0027,"},{"line_number":68,"context_line":"        )"},{"line_number":69,"context_line":"        return parser"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FHcg%3D","line":68,"in_reply_to":"AAAAM3%2F%2FHho%3D","updated":"2013-02-06 18:14:47.000000000","message":"I happen to disagree with Dolph that this is a problem.  It\u0027s basically a religious issue in that we both can point to precedence for our point of view and there isn\u0027t much of a technical reason either way.  If lines of code or maintenance is the reason we have much bigger fish to fry...","commit_id":"fbc412e533bd7cb07c6d930e194f660e14b2319f"}],"openstackclient/shell.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"e148e1880a166172682de5028e60768554b2acdb","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                    dest\u003d\u0027deferred_help\u0027,"},{"line_number":96,"context_line":"                    default\u003dFalse,"},{"line_number":97,"context_line":"                    help\u003d\"show this help message and exit\","},{"line_number":98,"context_line":"                )"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def build_option_parser(self, description, version):"},{"line_number":101,"context_line":"        parser \u003d super(OpenStackShell, self).build_option_parser("}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FLCg%3D","line":98,"updated":"2013-02-04 21:35:15.000000000","message":"What exactly would happen if you didn\u0027t add this? Just trying to get a better understanding of what\u0027s going on.","commit_id":"70c15bef268681aad1031faca94508a9aaa27e0d"}],"tests/common/test_commandmanager.py":[{"author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"change_message_id":"487343b1562bf4c0edde662e1f3c22d18b7483d6","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"class TestCommandManager(utils.TestCase):"},{"line_number":49,"context_line":"    def setUp(self):"},{"line_number":50,"context_line":"        super(TestCommandManager, self).setUp()"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    def test_add_command_group(self):"},{"line_number":53,"context_line":"        mgr \u003d FakeCommandManager(\u0027test\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FLPo%3D","line":50,"updated":"2013-02-04 19:46:12.000000000","message":"This setUp() method can go away, since nothing is happening here.","commit_id":"70c15bef268681aad1031faca94508a9aaa27e0d"},{"author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"change_message_id":"487343b1562bf4c0edde662e1f3c22d18b7483d6","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        mock_cmd_one \u003d mock.Mock()"},{"line_number":57,"context_line":"        mgr.add_command(\u0027mock\u0027, mock_cmd_one)"},{"line_number":58,"context_line":"        cmd_mock, name, args \u003d mgr.find_command([\u0027mock\u0027])"},{"line_number":59,"context_line":"        assert cmd_mock is mock_cmd_one"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        # Find a command added in initialization"},{"line_number":62,"context_line":"        cmd_one, name, args \u003d mgr.find_command([\u0027one\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FLP0%3D","line":59,"updated":"2013-02-04 19:46:12.000000000","message":"We want to use self.assertEqual() instead for these.","commit_id":"70c15bef268681aad1031faca94508a9aaa27e0d"},{"author":{"_account_id":209,"name":"Josh Kearney","email":"josh@jk0.org","username":"jk0"},"change_message_id":"74eed5e1ee7b71b15f9f58c7df52f81944a325c5","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    def test_add_command_group(self):"},{"line_number":50,"context_line":"        mgr \u003d FakeCommandManager(\u0027test\u0027)"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        # Make sure add_command() still functions"},{"line_number":53,"context_line":"        mock_cmd_one \u003d mock.Mock()"},{"line_number":54,"context_line":"        mgr.add_command(\u0027mock\u0027, mock_cmd_one)"},{"line_number":55,"context_line":"        cmd_mock, name, args \u003d mgr.find_command([\u0027mock\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FHKg%3D","line":52,"updated":"2013-02-06 20:22:13.000000000","message":"Should these be broken up into separate unit tests?","commit_id":"fbc412e533bd7cb07c6d930e194f660e14b2319f"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"757fe9c00e95899fc712ab1aa2ca05b95858dd4f","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    def test_add_command_group(self):"},{"line_number":50,"context_line":"        mgr \u003d FakeCommandManager(\u0027test\u0027)"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        # Make sure add_command() still functions"},{"line_number":53,"context_line":"        mock_cmd_one \u003d mock.Mock()"},{"line_number":54,"context_line":"        mgr.add_command(\u0027mock\u0027, mock_cmd_one)"},{"line_number":55,"context_line":"        cmd_mock, name, args \u003d mgr.find_command([\u0027mock\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FHFQ%3D","line":52,"in_reply_to":"AAAAM3%2F%2FHKg%3D","updated":"2013-02-06 21:13:07.000000000","message":"Maybe it is poorly named...we\u0027re really testing the new CommandManager._load_commands() here and the two methods that call it, one of which is an unchanged method in cliff.commandmanager.CommandMaanger.","commit_id":"fbc412e533bd7cb07c6d930e194f660e14b2319f"}]}