)]}'
{"neutronclient/neutron/v2_0/__init__.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"86339979e51e9aceb3691e0c32dca12e37ca0339","unresolved":false,"context_lines":[{"line_number":579,"context_line":"        if self.help_resource:"},{"line_number":580,"context_line":"            parser.add_argument("},{"line_number":581,"context_line":"                \u0027id\u0027, metavar\u003dself.resource.upper(),"},{"line_number":582,"context_line":"                help\u003dhelp_str % self.help_resource)"},{"line_number":583,"context_line":"        else:"},{"line_number":584,"context_line":"            parser.add_argument("},{"line_number":585,"context_line":"                \u0027id\u0027, metavar\u003dself.resource.upper(),"}],"source_content_type":"text/x-python","patch_set":8,"id":"da6ed579_7dba56c2","line":582,"updated":"2016-01-18 03:13:58.000000000","message":"The complete code can be rewritten as follows from L#579.\nWill save some Lines.\n\n if not self.help_resource:\n     self.help_resource \u003d self.resource\n parser.add_argument(\n     \u0027id\u0027, metavar\u003dself.resource.upper(),\n     help \u003d help_str %self.help_resource)","commit_id":"abcd2526ca74aeb96ec5a9e4b59c7648d6429b80"},{"author":{"_account_id":19383,"name":"linwei wu","email":"wlwwu@cn.ibm.com","username":"linwwu"},"change_message_id":"eb3938a686bb03d673ad2255d568849ef02a4455","unresolved":false,"context_lines":[{"line_number":579,"context_line":"        if self.help_resource:"},{"line_number":580,"context_line":"            parser.add_argument("},{"line_number":581,"context_line":"                \u0027id\u0027, metavar\u003dself.resource.upper(),"},{"line_number":582,"context_line":"                help\u003dhelp_str % self.help_resource)"},{"line_number":583,"context_line":"        else:"},{"line_number":584,"context_line":"            parser.add_argument("},{"line_number":585,"context_line":"                \u0027id\u0027, metavar\u003dself.resource.upper(),"}],"source_content_type":"text/x-python","patch_set":8,"id":"da6ed579_f85ad44f","line":582,"in_reply_to":"da6ed579_7dba56c2","updated":"2016-01-18 04:20:49.000000000","message":"yes. here, we can make it to be more clear. I will modify this","commit_id":"abcd2526ca74aeb96ec5a9e4b59c7648d6429b80"}],"neutronclient/neutron/v2_0/vpn/ikepolicy.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"86339979e51e9aceb3691e0c32dca12e37ca0339","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    \"\"\"Delete a given IKE policy.\"\"\""},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"    resource \u003d \u0027ikepolicy\u0027"},{"line_number":117,"context_line":"    help_resource \u003d \u0027IKE policy\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"da6ed579_ddc9025c","line":117,"updated":"2016-01-18 03:13:58.000000000","message":"+1","commit_id":"abcd2526ca74aeb96ec5a9e4b59c7648d6429b80"}],"neutronclient/neutron/v2_0/vpn/vpnservice.py":[{"author":{"_account_id":6659,"name":"Paul Michali","email":"pc@michali.net","username":"pcm"},"change_message_id":"632be2104307be25ceb41d3e23e1229183bb1582","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        if self.allow_names:"},{"line_number":97,"context_line":"            parser.add_argument("},{"line_number":98,"context_line":"                \u0027vpnservice\u0027, metavar\u003d\u0027VPNSERVICE\u0027,"},{"line_number":99,"context_line":"                help\u003d_(\u0027ID or name of Vpn service to delete.\u0027))"},{"line_number":100,"context_line":"        self.add_known_arguments(parser)"},{"line_number":101,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":4,"id":"da6ed579_f6818490","line":99,"range":{"start_line":99,"start_character":38,"end_line":99,"end_character":41},"updated":"2016-01-12 16:55:01.000000000","message":"nit: VPN","commit_id":"56ceeac3286a6aa4a191bac9af8388148db946e2"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"86339979e51e9aceb3691e0c32dca12e37ca0339","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    \"\"\"Delete a given VPN service.\"\"\""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    resource \u003d \u0027vpnservice\u0027"},{"line_number":92,"context_line":"    help_resource \u003d \u0027VPN service\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"da6ed579_bdc0ce2b","line":92,"updated":"2016-01-18 03:13:58.000000000","message":"I dont think change here is required.\nThe change was mainly require for IPSec because the protocol is names as thus.\n\nBoth VPN and vpn are clearly understandable by user.","commit_id":"abcd2526ca74aeb96ec5a9e4b59c7648d6429b80"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"4327484bd68a07940c1ec3a8a769c57238d4d875","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    \"\"\"Delete a given VPN service.\"\"\""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    resource \u003d \u0027vpnservice\u0027"},{"line_number":92,"context_line":"    help_resource \u003d \u0027VPN service\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"da6ed579_d3125358","line":92,"in_reply_to":"da6ed579_783584f5","updated":"2016-01-18 04:23:29.000000000","message":"I do not completely agree with this change, but lets see what other reviewers have to say about this.","commit_id":"abcd2526ca74aeb96ec5a9e4b59c7648d6429b80"},{"author":{"_account_id":19383,"name":"linwei wu","email":"wlwwu@cn.ibm.com","username":"linwwu"},"change_message_id":"eb3938a686bb03d673ad2255d568849ef02a4455","unresolved":false,"context_lines":[{"line_number":89,"context_line":"    \"\"\"Delete a given VPN service.\"\"\""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    resource \u003d \u0027vpnservice\u0027"},{"line_number":92,"context_line":"    help_resource \u003d \u0027VPN service\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"da6ed579_783584f5","line":92,"in_reply_to":"da6ed579_bdc0ce2b","updated":"2016-01-18 04:20:49.000000000","message":"I think we can modify this to make this more clear. this is similar like ikepolicy.","commit_id":"abcd2526ca74aeb96ec5a9e4b59c7648d6429b80"}],"neutronclient/shell.py":[{"author":{"_account_id":6659,"name":"Paul Michali","email":"pc@michali.net","username":"pcm"},"change_message_id":"ee2d51255d3122f8a87c474c46c9c3cbaca7ec1c","unresolved":false,"context_lines":[{"line_number":82,"context_line":"from neutronclient.neutron.v2_0.vpn import vpnservice"},{"line_number":83,"context_line":"from neutronclient.version import __version__"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"from oslo_serialization import jsonutils"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"VERSION \u003d \u00272.0\u0027"},{"line_number":88,"context_line":"NEUTRON_API_VERSION \u003d \u00272.0\u0027"}],"source_content_type":"text/x-python","patch_set":10,"id":"9a68dd71_613f3a26","line":85,"updated":"2016-01-20 13:47:57.000000000","message":"Unrelated change? Seems not used.","commit_id":"892ddc3e3ed201e19aad4b0dee20ba435632ff2f"},{"author":{"_account_id":19383,"name":"linwei wu","email":"wlwwu@cn.ibm.com","username":"linwwu"},"change_message_id":"04b3fc97319d72073644652e1eea75e4589b5e34","unresolved":false,"context_lines":[{"line_number":82,"context_line":"from neutronclient.neutron.v2_0.vpn import vpnservice"},{"line_number":83,"context_line":"from neutronclient.version import __version__"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"from oslo_serialization import jsonutils"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"VERSION \u003d \u00272.0\u0027"},{"line_number":88,"context_line":"NEUTRON_API_VERSION \u003d \u00272.0\u0027"}],"source_content_type":"text/x-python","patch_set":10,"id":"9a68dd71_7b87be61","line":85,"in_reply_to":"9a68dd71_613f3a26","updated":"2016-01-21 02:15:36.000000000","message":"i will post a new branch..","commit_id":"892ddc3e3ed201e19aad4b0dee20ba435632ff2f"}]}