)]}'
{"openstack/compute/v2/_proxy.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"7e4027db93eba41831724193920986d471f18f37","unresolved":false,"context_lines":[{"line_number":555,"context_line":"        security_group_id \u003d resource2.Resource._get_id(security_group)"},{"line_number":556,"context_line":"        server.remove_security_group(self.session, security_group_id)"},{"line_number":557,"context_line":""},{"line_number":558,"context_line":"    def add_server_fixed_ip(self, server, network_id):"},{"line_number":559,"context_line":"        \"\"\"Adds a fixed IP address to a server instance."},{"line_number":560,"context_line":""},{"line_number":561,"context_line":"        :param server: Either the ID of a server or a"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_97d6c4c1","line":558,"updated":"2016-11-30 19:16:35.000000000","message":"To stay with the previous conventions, this would be better as add_fixed_ip_to_server. The remove name would then be remove_fixed_ip_from_server, and the same convention would apply to the floating_ip methods as well.","commit_id":"57ff3b414468fe40700045be424711b999a6d507"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"03123c146e2a878c1f79c474ea964afddc97364b","unresolved":false,"context_lines":[{"line_number":569,"context_line":"        server.remove_security_group(self.session, security_group_id)"},{"line_number":570,"context_line":""},{"line_number":571,"context_line":"    def add_fixed_ip_to_server(self, server, network_id):"},{"line_number":572,"context_line":"        \"\"\"Adds a fixed IP address to a server instance."},{"line_number":573,"context_line":""},{"line_number":574,"context_line":"        :param server: Either the ID of a server or a"},{"line_number":575,"context_line":"                       :class:`~openstack.compute.v2.server.Server` instance."}],"source_content_type":"text/x-python","patch_set":2,"id":"3a71b18c_453a239a","line":572,"updated":"2016-12-06 16:46:53.000000000","message":"I think those pep8 failures are coming from the period at the end of this sentence and the other similar first-lines of the commit messages. I think if you remove it, the others messages it has should go away, because they sometimes cascade and you get 5 errors for one little thing.\n\nI think it\u0027d be more ideal to change that period than ignore the warnings so other things don\u0027t get in.","commit_id":"33a474b7461cc52ea4505b999cccbbe5fdf4b7c5"}],"tox.ini":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"4de2201e0cc60e8693cc1f5e979f9dcc26e0ee41","unresolved":false,"context_lines":[{"line_number":44,"context_line":"commands \u003d python setup.py build_sphinx"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"[flake8]"},{"line_number":47,"context_line":"ignore\u003dD100,D101,D102,D103,D104,D105,D200,D202,D204,D205,D211,D300,D301,D303,D400,D401"},{"line_number":48,"context_line":"show-source \u003d True"},{"line_number":49,"context_line":"exclude\u003d.venv,.git,.tox,dist,doc,*lib/python*,*egg,build"}],"source_content_type":"text/x-properties","patch_set":2,"id":"3a71b18c_8d7c2422","line":47,"updated":"2016-12-01 14:58:29.000000000","message":"Was this change intentional? It doesn\u0027t seem like any code you wrote would have added any sort of warnings to need to avoid, at least as far as I could see. Maybe the plus sign change?","commit_id":"33a474b7461cc52ea4505b999cccbbe5fdf4b7c5"},{"author":{"_account_id":8246,"name":"Qiming Teng","email":"tengqm@outlook.com","username":"tengqm"},"change_message_id":"d0e9d8ad2bebdc42fd68225db414ae0b7c81b05e","unresolved":false,"context_lines":[{"line_number":44,"context_line":"commands \u003d python setup.py build_sphinx"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"[flake8]"},{"line_number":47,"context_line":"ignore\u003dD100,D101,D102,D103,D104,D105,D200,D202,D204,D205,D211,D300,D301,D303,D400,D401"},{"line_number":48,"context_line":"show-source \u003d True"},{"line_number":49,"context_line":"exclude\u003d.venv,.git,.tox,dist,doc,*lib/python*,*egg,build"}],"source_content_type":"text/x-properties","patch_set":2,"id":"3a71b18c_d1d695c4","line":47,"in_reply_to":"3a71b18c_8d7c2422","updated":"2016-12-02 03:26:52.000000000","message":"not sure what happened to pep8, it is complaining about \"docstring for public method/class not provided, docstring without title line, ended with period ... \", all these boring stuff.\n\nMaybe it is good to ignore them at the moment.","commit_id":"33a474b7461cc52ea4505b999cccbbe5fdf4b7c5"}]}