)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"9cbbfdaff70a18afe097e3a4c7ce91ded486c3b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7d87546a_cf9e8a03","updated":"2022-03-18 16:24:46.000000000","message":"Brian, your name was on this so probably good to have you review.","commit_id":"840f0968a45e5ccdac3a49d30f5cfe08470c290c"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c31da12ea8ea4e6976955385c8050aba019bc0b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9d6a1f84_ff05f5dc","updated":"2022-03-29 19:15:29.000000000","message":"No objection to dropping the tags, but if we\u0027re going to have an image here, we shouldn\u0027t drop alt text for it.","commit_id":"840f0968a45e5ccdac3a49d30f5cfe08470c290c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"1adabed4ce84a6ab68f94276f0986d9cca818877","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"26b3626e_60865511","updated":"2022-03-18 17:06:11.000000000","message":"thanks, TC tag framework is going to remove so these doc update is needed.","commit_id":"840f0968a45e5ccdac3a49d30f5cfe08470c290c"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8f4fc3c0cc22b52b8add6146d9ccbc20fd00f55c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"18813370_6b19f7ac","updated":"2022-12-07 13:04:14.000000000","message":"LGTM.","commit_id":"ded97f6a729aed594727492ed35601e23708a7eb"}],"README.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c31da12ea8ea4e6976955385c8050aba019bc0b5","unresolved":true,"context_lines":[{"line_number":2,"context_line":"OpenStack Glance"},{"line_number":3,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":".. image:: https://governance.openstack.org/tc/badges/glance.svg"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Glance is an OpenStack project that provides services and associated libraries"},{"line_number":8,"context_line":"to store, browse, share, distribute and manage bootable disk images,"}],"source_content_type":"text/x-rst","patch_set":1,"id":"cff31641_d7b74a0f","line":5,"updated":"2022-03-29 19:15:29.000000000","message":"I think you should add an alt option to this image directive describing what\u0027s in the image, something like \"Glance is an OpenStack community project that follows CII best practices\".  If you don\u0027t want to do alt text, you could just delete the image, it doesn\u0027t bring much value (other than that if all the cool kids have badges and we don\u0027t, it makes us look bad).\n\nThis info is set on the badges when the SVG is generated [0], but it gets lost when the SVG is pulled in as an image \n\n[0] https://opendev.org/openstack/governance/commit/14e928bbf8d3de38b766160cdb9d1b4577b4c2b6","commit_id":"840f0968a45e5ccdac3a49d30f5cfe08470c290c"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"41e93cac598479185db0fd484dfb3d7a34e0102b","unresolved":true,"context_lines":[{"line_number":2,"context_line":"OpenStack Glance"},{"line_number":3,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":".. image:: https://governance.openstack.org/tc/badges/glance.svg"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Glance is an OpenStack project that provides services and associated libraries"},{"line_number":8,"context_line":"to store, browse, share, distribute and manage bootable disk images,"}],"source_content_type":"text/x-rst","patch_set":1,"id":"19cd6019_cb45c455","line":5,"in_reply_to":"cff31641_d7b74a0f","updated":"2022-03-29 19:28:01.000000000","message":"It doesn\u0027t matter to me one way or the other. I\u0027m not a believer that all images need alt tags, especially when they\u0027re abused for things like listing tags that really should be visible in text anyway. But happy to either remove or replace with descriptive text, whatever people want.","commit_id":"840f0968a45e5ccdac3a49d30f5cfe08470c290c"}]}