)]}'
{"openstack/resource2.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"82ad0d98826373b2062a1bddda00e31e586e1568","unresolved":false,"context_lines":[{"line_number":649,"context_line":"                 :data:`Resource.allow_update` is not set to ``True``."},{"line_number":650,"context_line":"        \"\"\""},{"line_number":651,"context_line":"        # Only try to update if we actually have anything to update."},{"line_number":652,"context_line":"        self._body.cleanse(\u0027id\u0027)"},{"line_number":653,"context_line":"        if not any([self._body.dirty, self._header.dirty]):"},{"line_number":654,"context_line":"            return self"},{"line_number":655,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_aab15991","line":652,"updated":"2017-02-22 14:48:50.000000000","message":"Can we just do the self._dirty.discard(\"id\") line directly here with a comment saying what it\u0027s for? What it\u0027s doing makes sense for an update, but I don\u0027t know that it needs to live as its own method since other calls aren\u0027t going to be stripping out IDs.","commit_id":"7eb9d10e4d7f2e8c7dc3785d1c6c7fd30ff7f95f"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"47da8499b235b9d34fe036249f8c5001651ac964","unresolved":false,"context_lines":[{"line_number":649,"context_line":"                 :data:`Resource.allow_update` is not set to ``True``."},{"line_number":650,"context_line":"        \"\"\""},{"line_number":651,"context_line":"        # Only try to update if we actually have anything to update."},{"line_number":652,"context_line":"        self._body.cleanse(\u0027id\u0027)"},{"line_number":653,"context_line":"        if not any([self._body.dirty, self._header.dirty]):"},{"line_number":654,"context_line":"            return self"},{"line_number":655,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_f0e8a412","line":652,"in_reply_to":"da36d5c6_aab15991","updated":"2017-02-22 15:11:34.000000000","message":"I can do that, but it would be self._body._dirty.discard(\"id\")","commit_id":"7eb9d10e4d7f2e8c7dc3785d1c6c7fd30ff7f95f"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"e59aa95c02c3aafeead954fe507ea468189a912c","unresolved":false,"context_lines":[{"line_number":649,"context_line":"                 :data:`Resource.allow_update` is not set to ``True``."},{"line_number":650,"context_line":"        \"\"\""},{"line_number":651,"context_line":"        # Only try to update if we actually have anything to update."},{"line_number":652,"context_line":"        self._body.cleanse(\u0027id\u0027)"},{"line_number":653,"context_line":"        if not any([self._body.dirty, self._header.dirty]):"},{"line_number":654,"context_line":"            return self"},{"line_number":655,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_4451b10d","line":652,"in_reply_to":"da36d5c6_f0e8a412","updated":"2017-02-22 15:23:25.000000000","message":"I think that\u0027d be fine just as long as we explain what it\u0027s there for.","commit_id":"7eb9d10e4d7f2e8c7dc3785d1c6c7fd30ff7f95f"}]}