)]}'
{"elastic_recheck/utils.py":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c8142c6d34019a11b855b1b4eadc901540902132","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    if filename:"},{"line_number":12,"context_line":"        with open(f\"{filename}.tmp\", \"w\") as f:"},{"line_number":13,"context_line":"            f.write(data)"},{"line_number":14,"context_line":"        f.close()"},{"line_number":15,"context_line":"        os.replace(f\"{filename}.tmp\", filename)"},{"line_number":16,"context_line":"    else:"},{"line_number":17,"context_line":"        sys.stdout.write(data)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f621f24_16448d73","line":14,"range":{"start_line":14,"start_character":8,"end_line":14,"end_character":17},"updated":"2020-11-09 05:20:26.000000000","message":"since we are already using with context manager. so we donot need to close the file, the context manager will take care of that","commit_id":"4bd55a35efe16bb5d4e34d04645eb3f1d5ed6ad9"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"85db42c266dd0e5f08747f56acaf53ebd832b92b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"import os"},{"line_number":2,"context_line":"import sys"},{"line_number":3,"context_line":"from typing import Optional"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1f621f24_15bbe271","line":1,"updated":"2020-11-09 10:18:36.000000000","message":"Do we want to copy the licence here? As it is a new file, all the other files contains it. It will keep the stuff consistent","commit_id":"5104f277b85e3716cdfc37f28be1f8aa594e28d5"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"413cac8cc855f762283cc5499ab2859654b16610","unresolved":false,"context_lines":[{"line_number":1,"context_line":"import os"},{"line_number":2,"context_line":"import sys"},{"line_number":3,"context_line":"from typing import Optional"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1f621f24_75617605","line":1,"in_reply_to":"1f621f24_15bbe271","updated":"2020-11-09 10:22:44.000000000","message":"No, it do not want to copy a license here. I find the header licenses as useful as SPAM. The only case where they are useful if for files that have different license than the\nrepository/folder hosting them.\n\nNext step would to add license header to all ansible files.","commit_id":"5104f277b85e3716cdfc37f28be1f8aa594e28d5"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fd1abb071471d903c895ef8887bb1312889293fc","unresolved":false,"context_lines":[{"line_number":1,"context_line":"import os"},{"line_number":2,"context_line":"import sys"},{"line_number":3,"context_line":"from typing import Optional"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1f621f24_f532a60f","line":1,"in_reply_to":"1f621f24_75617605","updated":"2020-11-09 10:25:39.000000000","message":"ok, thanks :-)","commit_id":"5104f277b85e3716cdfc37f28be1f8aa594e28d5"}]}