)]}'
{"ironic/api/app.py":[{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"bc88ed8731ec3c84cfc2661d3635e94cc232018e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# -*- coding: utf-8 -*-"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright © 2012 New Dream Network, LLC (DreamHost)"},{"line_number":4,"context_line":"# All Rights Reserved."},{"line_number":5,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_32eefa8b","line":2,"updated":"2014-06-19 01:39:03.000000000","message":"Should we keep same lines with others, that this line should have a beginning char \u0027#\u0027 here?\nThis comments can be applied to all rest files which has same issue.","commit_id":"45fb48b2f10b1224ea8b02a549049e773935b4dc"},{"author":{"_account_id":11638,"name":"Martin Geisler","email":"martin@geisler.net","username":"mg"},"change_message_id":"c3e611aa1b5db59a865ac2a9d4b501a2a2e8319d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# -*- coding: utf-8 -*-"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright © 2012 New Dream Network, LLC (DreamHost)"},{"line_number":4,"context_line":"# All Rights Reserved."},{"line_number":5,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_4e3d9a8d","line":2,"in_reply_to":"1ae5cdf2_32eefa8b","updated":"2014-06-19 07:45:55.000000000","message":"My little script (now available at https://github.com/mgeisler/clean-headers) has the logic that it tries to change as little as possible.\n\nSo it looks at the first 2 lines. If there is already a non-standard coding declaration (this is the case above), it normalizes it. If the second line is already a comment line or a pure whitespace, then it is left alone. When I browsed the OpenStack files (and other Python projects), then this seemed like the most common format: sometimes there is a blank line following the coding line, sometimes there is a \u0027#\u0027 line. The script tries to respect the current formatting with the idea that someone put it there on purpose.\n\nWould you like me to update the script so it rewrites the second line too in this case? The logic I\u0027m thinking of is this: change the second line from \u0027\u0027 (a blank line) to \u0027#\u0027 when it sees that the third line also starts with a \u0027#\u0027.","commit_id":"45fb48b2f10b1224ea8b02a549049e773935b4dc"},{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"18348f57b53a804c743d9abd454c7d7ae479231a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# -*- coding: utf-8 -*-"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright © 2012 New Dream Network, LLC (DreamHost)"},{"line_number":4,"context_line":"# All Rights Reserved."},{"line_number":5,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_b7607cd0","line":2,"in_reply_to":"1ae5cdf2_4e3d9a8d","updated":"2014-06-25 08:09:56.000000000","message":"It is not Mandatory for the lines have to be with \"#\" starting, so dont worry about this, just think about that we have the same behavior for all files.","commit_id":"45fb48b2f10b1224ea8b02a549049e773935b4dc"}],"ironic/tests/test_images.py":[{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"bc88ed8731ec3c84cfc2661d3635e94cc232018e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":"# -*- coding: utf-8 -*-"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"# Copyright 2013 Hewlett-Packard Development Company, L.P."}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_f2e01299","line":1,"updated":"2014-06-19 01:39:03.000000000","message":"Can we keep same line order that put the \u0027coding\u0027 line at first line?","commit_id":"45fb48b2f10b1224ea8b02a549049e773935b4dc"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"f99567b82e2cb9ebbfdf042745de5e4aef82d9a3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":"# -*- coding: utf-8 -*-"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"# Copyright 2013 Hewlett-Packard Development Company, L.P."}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_31798f93","line":1,"in_reply_to":"1ae5cdf2_8ef882c5","updated":"2014-06-19 08:07:45.000000000","message":"# Vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4  \n\nmust be deleted per global OpenStack policy","commit_id":"45fb48b2f10b1224ea8b02a549049e773935b4dc"},{"author":{"_account_id":11638,"name":"Martin Geisler","email":"martin@geisler.net","username":"mg"},"change_message_id":"c3e611aa1b5db59a865ac2a9d4b501a2a2e8319d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":"# -*- coding: utf-8 -*-"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"# Copyright 2013 Hewlett-Packard Development Company, L.P."}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_8ef882c5","line":1,"in_reply_to":"1ae5cdf2_f2e01299","updated":"2014-06-19 07:45:55.000000000","message":"Interesting case for my script :-)\n\nSo, as you probably know, Python allows for the coding line to be the second line in the file. This is to accommodate a possible\n\n  #!/usr/bin/python\n\nline as the first line. My script therefore looks for a coding line in the first two lines. If it finds a coding line there, it will normalize it as you see here -- but it doesn\u0027t try to reorder things.\n\nI agree that it could be nice to see the lines reordered, so I\u0027ll add that option to the script.","commit_id":"45fb48b2f10b1224ea8b02a549049e773935b4dc"}]}