)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"f6880afb99458dc3b4350298d90c50f36301d732","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add weblate users"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"fix Zuul build failure"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Added weblate API codes to get ready for"},{"line_number":12,"context_line":"Zanata to Weblate migration"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"d2e9dcdc_b9915206","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":22},"updated":"2023-11-10 15:49:16.000000000","message":"Is this related to the patch?","commit_id":"2058b7fc8d7e6866d26e45ee600954147dc56b38"},{"author":{"_account_id":36160,"name":"Sion Shin","display_name":"sion","email":"kmicety1@gmail.com","username":"sion"},"change_message_id":"70a3846b66d9962fda975e53d67abc93513c70b3","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add weblate users"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"fix Zuul build failure"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Added weblate API codes to get ready for"},{"line_number":12,"context_line":"Zanata to Weblate migration"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"6b62acea_da4e8560","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":22},"in_reply_to":"d2e9dcdc_b9915206","updated":"2023-11-13 06:52:29.000000000","message":"I just fixed commit message.","commit_id":"2058b7fc8d7e6866d26e45ee600954147dc56b38"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"f6880afb99458dc3b4350298d90c50f36301d732","unresolved":true,"context_lines":[{"line_number":11,"context_line":"Added weblate API codes to get ready for"},{"line_number":12,"context_line":"Zanata to Weblate migration"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Requires WeblateUtils.py from #893361"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Implements: blueprint weblate-tools"},{"line_number":17,"context_line":"Change-Id: I8d1a2393cc50541a29bd9581c4ac2574b9dc9813"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"16925a9c_080c2cd9","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":37},"updated":"2023-11-10 15:49:16.000000000","message":"I think you don\u0027t have to mention this - any reason you specified on this commit message?","commit_id":"2058b7fc8d7e6866d26e45ee600954147dc56b38"},{"author":{"_account_id":36160,"name":"Sion Shin","display_name":"sion","email":"kmicety1@gmail.com","username":"sion"},"change_message_id":"70a3846b66d9962fda975e53d67abc93513c70b3","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Added weblate API codes to get ready for"},{"line_number":12,"context_line":"Zanata to Weblate migration"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Requires WeblateUtils.py from #893361"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Implements: blueprint weblate-tools"},{"line_number":17,"context_line":"Change-Id: I8d1a2393cc50541a29bd9581c4ac2574b9dc9813"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3bfc9b41_f0848628","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":37},"in_reply_to":"16925a9c_080c2cd9","updated":"2023-11-13 06:52:29.000000000","message":"Yes. Since weblate_utils.py already has been merged, I removed this from commit message.","commit_id":"2058b7fc8d7e6866d26e45ee600954147dc56b38"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"86a4928f669dcf9802131ee3f15dbc0d7c90121f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a93a2654_e5bf5cc9","updated":"2023-09-22 16:20:30.000000000","message":"893361 now does not have WeblateUtils.py - will review again once WeblateUtils.py is merged and please let me know if then","commit_id":"742a453dc3ad572cf1b7e67738ee3ebceaa0abe9"},{"author":{"_account_id":36160,"name":"Sion Shin","display_name":"sion","email":"kmicety1@gmail.com","username":"sion"},"change_message_id":"72126cd79d9be69f914c070adb6923c2013b165e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"48e67c2e_3e177f2e","in_reply_to":"a93a2654_e5bf5cc9","updated":"2023-11-04 05:19:55.000000000","message":"Okay. I\u0027ll let you know when weblate_utils.py is merged. Thank you for reviewing.","commit_id":"742a453dc3ad572cf1b7e67738ee3ebceaa0abe9"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"da95fa6defbeca54e633a408ae7392c3c2142f9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"855d410c_63c50a38","updated":"2023-10-27 18:50:13.000000000","message":"Thank you - once you rebase on the top of this patch: https://review.opendev.org/c/openstack/i18n/+/896308 , I will do more detailed review.","commit_id":"8b4dee713acf258c4451155f7c9a255f47cdc0a9"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"f6880afb99458dc3b4350298d90c50f36301d732","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a220ba83_f5642652","updated":"2023-11-10 15:49:16.000000000","message":"(Note: I am welcoming your peer team members\u0027 review thank you all)","commit_id":"2058b7fc8d7e6866d26e45ee600954147dc56b38"},{"author":{"_account_id":36160,"name":"Sion Shin","display_name":"sion","email":"kmicety1@gmail.com","username":"sion"},"change_message_id":"100e763b9d3b462d3f887377c3e72fa7d8719042","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8a6c0e20_8fd5b3e6","updated":"2023-11-07 14:29:45.000000000","message":"If I modify weblate_utils.py a bit, we call the static string only once as you said, but I think that would require modifying the other files that were previously calling the static string as well.","commit_id":"2058b7fc8d7e6866d26e45ee600954147dc56b38"},{"author":{"_account_id":36160,"name":"Sion Shin","display_name":"sion","email":"kmicety1@gmail.com","username":"sion"},"change_message_id":"99cdd37dfa457e59ca9916b41cfd1cd2ad3a504f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"77f03520_363b90af","in_reply_to":"3f2d66be_9b033f01","updated":"2023-11-14 09:42:10.000000000","message":"Done","commit_id":"2058b7fc8d7e6866d26e45ee600954147dc56b38"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"f6880afb99458dc3b4350298d90c50f36301d732","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3f2d66be_9b033f01","in_reply_to":"8a6c0e20_8fd5b3e6","updated":"2023-11-10 15:49:16.000000000","message":"Thx for sharing your opinion - we can more discuss and change later.","commit_id":"2058b7fc8d7e6866d26e45ee600954147dc56b38"},{"author":{"_account_id":23279,"name":"Seongsoo Cho","display_name":"Seongsoo Cho","email":"ppiyakk2@printf.kr","username":"seongsoo.cho"},"change_message_id":"ce86de46e4a19610759eb6981f81429ab1637471","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5f20087b_678c56dc","updated":"2024-01-16 15:28:41.000000000","message":"It seems like that there are no more feedback about this patch, let\u0027s merge this first.","commit_id":"02b0574557982bb723abc932acef826857d96e17"}],"tools/weblate/weblate_users.py":[{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"da95fa6defbeca54e633a408ae7392c3c2142f9b","unresolved":true,"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":"import argparse"},{"line_number":17,"context_line":"from collections import OrderedDict"},{"line_number":18,"context_line":"import json"},{"line_number":19,"context_line":"import logging"},{"line_number":20,"context_line":"import os"},{"line_number":21,"context_line":"import sys"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from WeblateUtils import IniConfig"},{"line_number":24,"context_line":"from WeblateUtils import WeblateRestService"},{"line_number":25,"context_line":"import yaml"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"WEBLATE_URI \u003d \"https://openstack.weblate.cloud/api/%s\""},{"line_number":28,"context_line":"LOG \u003d logging.getLogger(\"weblate_users\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"ab8f315d_27844069","line":25,"range":{"start_line":16,"start_character":0,"end_line":25,"end_character":11},"updated":"2023-10-27 18:50:13.000000000","message":"Please address to follow with OpenStack guideline on this part - you can see the comments on https://review.opendev.org/c/openstack/i18n/+/896308 and this doc: https://docs.openstack.org/hacking/latest/user/hacking.html#imports","commit_id":"8b4dee713acf258c4451155f7c9a255f47cdc0a9"},{"author":{"_account_id":36160,"name":"Sion Shin","display_name":"sion","email":"kmicety1@gmail.com","username":"sion"},"change_message_id":"800493dfedfa68ce24e103ebd71b8183d78ade05","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":"import argparse"},{"line_number":17,"context_line":"from collections import OrderedDict"},{"line_number":18,"context_line":"import json"},{"line_number":19,"context_line":"import logging"},{"line_number":20,"context_line":"import os"},{"line_number":21,"context_line":"import sys"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from WeblateUtils import IniConfig"},{"line_number":24,"context_line":"from WeblateUtils import WeblateRestService"},{"line_number":25,"context_line":"import yaml"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"WEBLATE_URI \u003d \"https://openstack.weblate.cloud/api/%s\""},{"line_number":28,"context_line":"LOG \u003d logging.getLogger(\"weblate_users\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"e2d2d201_5472a312","line":25,"range":{"start_line":16,"start_character":0,"end_line":25,"end_character":11},"in_reply_to":"ab8f315d_27844069","updated":"2023-11-04 05:27:04.000000000","message":"Done","commit_id":"8b4dee713acf258c4451155f7c9a255f47cdc0a9"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"da95fa6defbeca54e633a408ae7392c3c2142f9b","unresolved":true,"context_lines":[{"line_number":138,"context_line":"if __name__ \u003d\u003d \"__main__\":"},{"line_number":139,"context_line":"    # Loads weblate.ini configuration file"},{"line_number":140,"context_line":"    try:"},{"line_number":141,"context_line":"        wc \u003d IniConfig(os.path.expanduser(\u0027~/.config/weblate.ini\u0027))"},{"line_number":142,"context_line":"    except ValueError as e:"},{"line_number":143,"context_line":"        sys.exit(e)"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"961be112_a2450b3f","line":141,"range":{"start_line":141,"start_character":43,"end_line":141,"end_character":64},"updated":"2023-10-27 18:50:13.000000000","message":"Question: would it be a good idea if this rather static string is specified in WeblateUtils, not here and other implementation source code files?","commit_id":"8b4dee713acf258c4451155f7c9a255f47cdc0a9"},{"author":{"_account_id":36160,"name":"Sion Shin","display_name":"sion","email":"kmicety1@gmail.com","username":"sion"},"change_message_id":"800493dfedfa68ce24e103ebd71b8183d78ade05","unresolved":false,"context_lines":[{"line_number":138,"context_line":"if __name__ \u003d\u003d \"__main__\":"},{"line_number":139,"context_line":"    # Loads weblate.ini configuration file"},{"line_number":140,"context_line":"    try:"},{"line_number":141,"context_line":"        wc \u003d IniConfig(os.path.expanduser(\u0027~/.config/weblate.ini\u0027))"},{"line_number":142,"context_line":"    except ValueError as e:"},{"line_number":143,"context_line":"        sys.exit(e)"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"f9786f8d_ac1f9377","line":141,"range":{"start_line":141,"start_character":43,"end_line":141,"end_character":64},"in_reply_to":"157a5783_15798f14","updated":"2023-11-04 05:27:04.000000000","message":"Done","commit_id":"8b4dee713acf258c4451155f7c9a255f47cdc0a9"},{"author":{"_account_id":36160,"name":"Sion Shin","display_name":"sion","email":"kmicety1@gmail.com","username":"sion"},"change_message_id":"72126cd79d9be69f914c070adb6923c2013b165e","unresolved":true,"context_lines":[{"line_number":138,"context_line":"if __name__ \u003d\u003d \"__main__\":"},{"line_number":139,"context_line":"    # Loads weblate.ini configuration file"},{"line_number":140,"context_line":"    try:"},{"line_number":141,"context_line":"        wc \u003d IniConfig(os.path.expanduser(\u0027~/.config/weblate.ini\u0027))"},{"line_number":142,"context_line":"    except ValueError as e:"},{"line_number":143,"context_line":"        sys.exit(e)"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"157a5783_15798f14","line":141,"range":{"start_line":141,"start_character":43,"end_line":141,"end_character":64},"in_reply_to":"961be112_a2450b3f","updated":"2023-11-04 05:19:55.000000000","message":"zanata_users.py has a static string of the same format, so I think it\u0027s okay to keep this format.","commit_id":"8b4dee713acf258c4451155f7c9a255f47cdc0a9"}]}