)]}'
{"setup.py":[{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"9fe1bca46f8857f6127a45fd863bcb2bffb7db70","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# THIS FILE IS MANAGED BY THE GLOBAL REQUIREMENTS REPO - DO NOT EDIT"},{"line_number":17,"context_line":"import setuptools"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# In python \u003c 2.7.4, a lazy loading of package `pbr` will break"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_2d62df17","line":16,"updated":"2019-07-24 16:19:31.000000000","message":"Either this comment is wrong, or we should not be editing setup.py","commit_id":"bcc287da16ec83f34afafa544c9247d7e9d06ddd"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"01b17c157fedf8bb3fdfa451f7fa3a4be47c2d5b","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# THIS FILE IS MANAGED BY THE GLOBAL REQUIREMENTS REPO - DO NOT EDIT"},{"line_number":17,"context_line":"import setuptools"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# In python \u003c 2.7.4, a lazy loading of package `pbr` will break"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_690d1634","line":16,"in_reply_to":"7faddb67_2d62df17","updated":"2019-07-25 17:29:08.000000000","message":"Not sure to understand why you put a `-1` here, this isn\u0027t my changes.\n\nCan you give more info to more understand the situation and your comment.","commit_id":"bcc287da16ec83f34afafa544c9247d7e9d06ddd"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"3245a9c04455218ff9f3ea31007a3f84fdf1cd01","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# THIS FILE IS MANAGED BY THE GLOBAL REQUIREMENTS REPO - DO NOT EDIT"},{"line_number":17,"context_line":"import setuptools"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# In python \u003c 2.7.4, a lazy loading of package `pbr` will break"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_8e78a3f8","line":16,"in_reply_to":"7faddb67_2d62df17","updated":"2019-07-25 06:27:20.000000000","message":"The comment is not correct anymore, syncing stopped over a year ago as part of requirements sync","commit_id":"bcc287da16ec83f34afafa544c9247d7e9d06ddd"},{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"53d0a64529214223948616824e7e084995e3dda7","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# THIS FILE IS MANAGED BY THE GLOBAL REQUIREMENTS REPO - DO NOT EDIT"},{"line_number":17,"context_line":"import setuptools"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# In python \u003c 2.7.4, a lazy loading of package `pbr` will break"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_eb672524","line":16,"in_reply_to":"7faddb67_690d1634","updated":"2019-08-05 21:34:08.000000000","message":"Hi Hervé,  I originally reviewed this with a -1 because this file has a comment that says \"DO NOT EDIT\" and this patch makes a change to the file.\n\nI was not aware the sync job that prevented this file from being edited was no longer used.  As Andreas pointed out, the comment is wrong and it is now OK to edit this file.","commit_id":"bcc287da16ec83f34afafa544c9247d7e9d06ddd"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"3245a9c04455218ff9f3ea31007a3f84fdf1cd01","unresolved":false,"context_lines":[{"line_number":27,"context_line":"setuptools.setup("},{"line_number":28,"context_line":"    setup_requires\u003d[\u0027pbr\u003e\u003d2.0.0\u0027],"},{"line_number":29,"context_line":"    pbr\u003dTrue,"},{"line_number":30,"context_line":"    long_description_content_type\u003d\u0027text/markdown\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_4eb70b88","line":30,"updated":"2019-07-25 06:27:20.000000000","message":"README file is README.md - changing it to README.rst would help as well like the rest of OpenStack does. But this looks fine...","commit_id":"bcc287da16ec83f34afafa544c9247d7e9d06ddd"}]}
