)]}'
{"ci/roles/keystone_idp/tasks/main.yml":[{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"54a27a9e80afdcfc738049ca18f95775005f399b","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    # explode if state is absent and the identity provider does not exist"},{"line_number":29,"context_line":"    - name: \"Ensure IDP doesn\u0027t exist to start\""},{"line_number":30,"context_line":"      openstack.cloud.federation_idp:"},{"line_number":31,"context_line":"        state: absent"},{"line_number":32,"context_line":"        name: \u0027ansible_identity_provider\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    # \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"5ffecac8_ab3ba0b8","line":31,"updated":"2023-01-28 11:17:00.000000000","message":"what is the sense of dropping quotes in some places while introducing them in other places? Looks not really consistent","commit_id":"90b110794f8ee9905c48ce7c524bcbb46e53ec74"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"03e8eca8813dd98d5469ba3cbabaa12d8076e199","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    # explode if state is absent and the identity provider does not exist"},{"line_number":29,"context_line":"    - name: \"Ensure IDP doesn\u0027t exist to start\""},{"line_number":30,"context_line":"      openstack.cloud.federation_idp:"},{"line_number":31,"context_line":"        state: absent"},{"line_number":32,"context_line":"        name: \u0027ansible_identity_provider\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    # \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3aa75f72_cae89ca8","line":31,"in_reply_to":"5ffecac8_ab3ba0b8","updated":"2023-01-28 12:10:19.000000000","message":"Agree, it should be consistent. In this file all those quotes for \u0027name\u0027 attributes are left overs from string-replace operations. Due to time pressure I will leave the \"Being consistent with strings quoting\" as future work :/","commit_id":"90b110794f8ee9905c48ce7c524bcbb46e53ec74"}],"plugins/modules/federation_idp.py":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"3267c3ee2bb185021849193f3ff62f86a34c9d46","unresolved":true,"context_lines":[{"line_number":140,"context_line":""},{"line_number":141,"context_line":"        if identity_provider is None:"},{"line_number":142,"context_line":"            self.exit_json(changed\u003dis_changed)"},{"line_number":143,"context_line":"        else:"},{"line_number":144,"context_line":"            self.exit_json("},{"line_number":145,"context_line":"                changed\u003dis_changed,"},{"line_number":146,"context_line":"                identity_provider\u003didentity_provider.to_dict(computed\u003dFalse))"}],"source_content_type":"text/x-python","patch_set":9,"id":"27bb81d2_30c84255","line":143,"range":{"start_line":143,"start_character":8,"end_line":143,"end_character":13},"updated":"2023-01-28 12:08:29.000000000","message":"\"else\" is not needed here","commit_id":"90b110794f8ee9905c48ce7c524bcbb46e53ec74"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"48288f9c4ebfde3b62bfa652ca5c9a0c89e2ff24","unresolved":false,"context_lines":[{"line_number":140,"context_line":""},{"line_number":141,"context_line":"        if identity_provider is None:"},{"line_number":142,"context_line":"            self.exit_json(changed\u003dis_changed)"},{"line_number":143,"context_line":"        else:"},{"line_number":144,"context_line":"            self.exit_json("},{"line_number":145,"context_line":"                changed\u003dis_changed,"},{"line_number":146,"context_line":"                identity_provider\u003didentity_provider.to_dict(computed\u003dFalse))"}],"source_content_type":"text/x-python","patch_set":9,"id":"c8237746_ab0e344a","line":143,"range":{"start_line":143,"start_character":8,"end_line":143,"end_character":13},"in_reply_to":"27bb81d2_30c84255","updated":"2023-01-28 12:11:04.000000000","message":"Its not needed but spelled out explicitly here for symmetry.","commit_id":"90b110794f8ee9905c48ce7c524bcbb46e53ec74"}]}