)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37213,"name":"ahnhyun","display_name":"ahnhyun","email":"ahnsunny926@naver.com","username":"ahnhyun"},"change_message_id":"b95f05cec1bf43c265d1da8db1f397c4741fb2bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"066e23f8_b3f2fccc","updated":"2025-06-02 02:19:52.000000000","message":"recheck","commit_id":"36c584793633f19be369b38b1d4c8f24e846825f"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"f797e069e9e9bd50ae706d4179fca9865d4d8483","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"204d32b3_5a43e9e1","updated":"2026-03-13 16:55:50.000000000","message":"recheck Thanks for working on this. I left a few comments","commit_id":"36c584793633f19be369b38b1d4c8f24e846825f"}],"designateclient/v2/cli/limits.py":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"f797e069e9e9bd50ae706d4179fca9865d4d8483","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright 2014 Hewlett-Packard Development Company, L.P."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Author: Endre Karlson \u003cendre.karlson@hp.com\u003e"},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":6,"context_line":"# not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":4,"id":"566115e5_c0cdffce","line":3,"range":{"start_line":1,"start_character":0,"end_line":3,"end_character":46},"updated":"2026-03-13 16:55:50.000000000","message":"We should probably remove these lines","commit_id":"36c584793633f19be369b38b1d4c8f24e846825f"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"f797e069e9e9bd50ae706d4179fca9865d4d8483","unresolved":true,"context_lines":[{"line_number":34,"context_line":"        common.set_all_common_headers(client, parsed_args)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        data \u003d client.limits.get()"},{"line_number":37,"context_line":"        limits \u003d data.get(\u0027limits\u0027, data)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        limits_list \u003d [(key, str(value)) for key, value in limits.items()]"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        return self.columns, limits_list"}],"source_content_type":"text/x-python","patch_set":4,"id":"cd6734b2_5b570d21","line":41,"range":{"start_line":37,"start_character":8,"end_line":41,"end_character":40},"updated":"2026-03-13 16:55:50.000000000","message":"The API returns a flat dict, there\u0027s no limits wrapper key, so this always falls back to data. We can just use data directly.\nSay,\n```\nreturn self.columns, sorted(data.items())\n```","commit_id":"36c584793633f19be369b38b1d4c8f24e846825f"}],"designateclient/v2/limits.py":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"f797e069e9e9bd50ae706d4179fca9865d4d8483","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"268098ed_78a019b0","line":27,"range":{"start_line":23,"start_character":4,"end_line":27,"end_character":0},"updated":"2026-03-13 16:55:50.000000000","message":"The list() method isn\u0027t used anywhere — the CLI calls get(). The limits API endpoint (GET /limits) returns a flat dict, not a paginated collection, so criterion/marker/limit params don\u0027t apply here. We should remove these lines.","commit_id":"36c584793633f19be369b38b1d4c8f24e846825f"}],"setup.cfg":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"f797e069e9e9bd50ae706d4179fca9865d4d8483","unresolved":true,"context_lines":[{"line_number":106,"context_line":"    tsigkey_set \u003d designateclient.v2.cli.tsigkeys:SetTSIGKeyCommand"},{"line_number":107,"context_line":"    tsigkey_delete \u003d designateclient.v2.cli.tsigkeys:DeleteTSIGKeyCommand"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    dns_limit_list \u003d designateclient.v2.cli.limits:ListLimitsCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":4,"id":"90d44939_8cd0b160","line":109,"updated":"2026-03-13 16:55:50.000000000","message":"As the line at the bottom suggests - this patch is removing the newline at the end of this file","commit_id":"36c584793633f19be369b38b1d4c8f24e846825f"}]}