)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"29a9a774decf75f520de328e70f42f979ec2321b","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This allows a caller to pass None for the project_id if it only wants"},{"line_number":10,"context_line":"it to check the registered limit for a given resource. This is useful"},{"line_number":11,"context_line":"for non-project-scoped resourced where we just want to make sure some"},{"line_number":12,"context_line":"global limit hasn\u0027t been exceeded. This would also be relevant for"},{"line_number":13,"context_line":"resources that are created by system-scoped users, such as host"},{"line_number":14,"context_line":"aggregates."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"2330597f_794f2a61","line":11,"range":{"start_line":11,"start_character":23,"end_line":11,"end_character":32},"updated":"2022-01-24 17:20:25.000000000","message":"resources","commit_id":"7e4f36abdb72cea991b1b99a67d30b088cc60d7d"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"29a9a774decf75f520de328e70f42f979ec2321b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"66db7470_7b7a72b9","updated":"2022-01-24 17:20:25.000000000","message":"Some nits. Nothing blocking","commit_id":"7e4f36abdb72cea991b1b99a67d30b088cc60d7d"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"593c4e484bcdba29f801fdce2fb091376b0c0092","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3e7f9c7d_c1a553aa","updated":"2022-01-21 22:22:22.000000000","message":"Still LGTM, I just removed WIP from the commit message","commit_id":"7e4f36abdb72cea991b1b99a67d30b088cc60d7d"}],"oslo_limit/limit.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"29a9a774decf75f520de328e70f42f979ec2321b","unresolved":true,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"        \"\"\""},{"line_number":126,"context_line":"        if project_id is not None and ("},{"line_number":127,"context_line":"                not project_id or not isinstance(project_id, str)):"},{"line_number":128,"context_line":"            msg \u003d \u0027project_id must be a non-empty string or None.\u0027"},{"line_number":129,"context_line":"            raise ValueError(msg)"},{"line_number":130,"context_line":"        if not isinstance(deltas, dict) or len(deltas) \u003d\u003d 0:"}],"source_content_type":"text/x-python","patch_set":3,"id":"eadb9aab_4450863b","line":127,"range":{"start_line":127,"start_character":16,"end_line":127,"end_character":65},"updated":"2022-01-24 17:20:25.000000000","message":"nit: I\u0027d probably rework this to check for an explicit empty string now that we\u0027re accepting None. It looks slightly weird as-is (IMO)\n\n  if project_id is not None and (\n      not isinstance(project_id, str) or project_id \u003d\u003d \u0027\u0027\n  ):","commit_id":"7e4f36abdb72cea991b1b99a67d30b088cc60d7d"}]}