)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9683cff1008099c14eb34224747f576bf62d4b68","unresolved":true,"context_lines":[{"line_number":7,"context_line":"ruff: Enable B, RUF rules"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The B (flake8-bugbear) and RUF (ruff) rule sets are enabled in the"},{"line_number":10,"context_line":"default ruff rules in 0.16.0. They\u0027re useful and it make sense to enable"},{"line_number":11,"context_line":"them here also."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I478baf973eef7c45acd626f2d5433446ab62977d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"93802645_d46523ce","line":10,"range":{"start_line":10,"start_character":52,"end_line":10,"end_character":56},"updated":"2026-09-30 08:19:29.000000000","message":"makes?","commit_id":"ac7ead2abc3372a1cfdb4add549cd6e6288ddd11"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f332fa2532e8117b44f6e151eaec071e750625fd","unresolved":false,"context_lines":[{"line_number":7,"context_line":"ruff: Enable B, RUF rules"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The B (flake8-bugbear) and RUF (ruff) rule sets are enabled in the"},{"line_number":10,"context_line":"default ruff rules in 0.16.0. They\u0027re useful and it make sense to enable"},{"line_number":11,"context_line":"them here also."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I478baf973eef7c45acd626f2d5433446ab62977d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a97c6123_c5ba353a","line":10,"range":{"start_line":10,"start_character":52,"end_line":10,"end_character":56},"in_reply_to":"93802645_d46523ce","updated":"2026-10-06 15:58:08.000000000","message":"Done","commit_id":"ac7ead2abc3372a1cfdb4add549cd6e6288ddd11"}],"openstack_requirements/cmds/check_exists.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9683cff1008099c14eb34224747f576bf62d4b68","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"\"\"\"Check for a package dependency."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Check to see if a package from a project\u0027s dependencies exist in g-r or u-c."},{"line_number":16,"context_line":"\"\"\""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import argparse"}],"source_content_type":"text/x-python","patch_set":1,"id":"65ffb610_ae50cb37","line":15,"range":{"start_line":15,"start_character":56,"end_line":15,"end_character":61},"updated":"2026-09-30 08:19:29.000000000","message":"```suggestion\nCheck to see if a package from a project\u0027s dependencies exists in g-r or u-c.\n```","commit_id":"ac7ead2abc3372a1cfdb4add549cd6e6288ddd11"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f332fa2532e8117b44f6e151eaec071e750625fd","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"\"\"\"Check for a package dependency."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Check to see if a package from a project\u0027s dependencies exist in g-r or u-c."},{"line_number":16,"context_line":"\"\"\""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import argparse"}],"source_content_type":"text/x-python","patch_set":1,"id":"ec14029f_779f5567","line":15,"range":{"start_line":15,"start_character":56,"end_line":15,"end_character":61},"in_reply_to":"65ffb610_ae50cb37","updated":"2026-10-06 15:58:08.000000000","message":"Done","commit_id":"ac7ead2abc3372a1cfdb4add549cd6e6288ddd11"}],"openstack_requirements/tests/test_check.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9683cff1008099c14eb34224747f576bf62d4b68","unresolved":true,"context_lines":[{"line_number":306,"context_line":""},{"line_number":307,"context_line":"class TestGetExclusions(testtools.TestCase):"},{"line_number":308,"context_line":"    def test_none(self):"},{"line_number":309,"context_line":"        req \u003d next(iter(check.get_global_reqs(\u0027name\u003e\u003d1.2\u0027)[\u0027name\u0027]))"},{"line_number":310,"context_line":"        self.assertEqual("},{"line_number":311,"context_line":"            set(),"},{"line_number":312,"context_line":"            check._get_exclusions(req),"}],"source_content_type":"text/x-python","patch_set":1,"id":"9e916bbe_6210a460","line":309,"updated":"2026-09-30 08:19:29.000000000","message":"this one I actually object to. I can see where this is coming from in terms of trying to optimize performance (or so I think), but it makes the test much more difficult to understand IMO","commit_id":"ac7ead2abc3372a1cfdb4add549cd6e6288ddd11"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d4c61556b6be7ee700dc569c45e08e3ec2588a0a","unresolved":false,"context_lines":[{"line_number":306,"context_line":""},{"line_number":307,"context_line":"class TestGetExclusions(testtools.TestCase):"},{"line_number":308,"context_line":"    def test_none(self):"},{"line_number":309,"context_line":"        req \u003d next(iter(check.get_global_reqs(\u0027name\u003e\u003d1.2\u0027)[\u0027name\u0027]))"},{"line_number":310,"context_line":"        self.assertEqual("},{"line_number":311,"context_line":"            set(),"},{"line_number":312,"context_line":"            check._get_exclusions(req),"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff310dbc_fd53fe9d","line":309,"in_reply_to":"9e916bbe_6210a460","updated":"2026-09-30 09:57:22.000000000","message":"I agree with this. Let me yank it out and disable the rule.","commit_id":"ac7ead2abc3372a1cfdb4add549cd6e6288ddd11"}]}
