)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"55d20e63d1881c5a06b35ea0f88b7d47ef92cfc2","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     cid \u003cafonnepaulc@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-09-20 17:09:41 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Bump sphinx version to fix deprecation warning."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #2078361"},{"line_number":10,"context_line":"Change-Id: I3aa3f2f87b38c3ce88414e9c52cbe712eb98cf8e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3d6cc622_4ed5fbb3","line":7,"updated":"2024-10-10 07:43:08.000000000","message":"Maybe you can a bit more detail in the commit message.","commit_id":"360ae5d4b6078755f94a3bb1592a82ceab65003c"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"60a4a896395be9549f2a1b2931dce92ea98da0e7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f343ab3c_7d499192","updated":"2024-09-20 15:17:06.000000000","message":"It seems that sphinx supports pathlib.Path since 7.2.0 . Should we bump the min version (and remove that logic to support sphinx \u003c 7) ?\n\nhttps://github.com/sphinx-doc/sphinx/commit/49d830467098cc149390bc8349c19b7501e0920f","commit_id":"f82e2ffc9736516e1c4534c2bd9ee78099c00222"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d67f1845f4268dda0d92900b831d8b71f77e6d2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6a4e90dd_7fddc1a5","in_reply_to":"00415ac5_7de7ad3e","updated":"2024-09-23 12:20:17.000000000","message":"We don\u0027t usually add a release note for dependency bump so I think we can skip adding one.","commit_id":"f82e2ffc9736516e1c4534c2bd9ee78099c00222"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"cf499307819c96555144e3cd6d71aa2bb27d7f18","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"00415ac5_7de7ad3e","in_reply_to":"f343ab3c_7d499192","updated":"2024-09-20 16:11:08.000000000","message":"That\u0027s much more straightforward. Do we need a release note for this?","commit_id":"f82e2ffc9736516e1c4534c2bd9ee78099c00222"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"0597dacb0dae7bfe107295944b8cdac5a3b582f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7e7d5dcf_1a416dc0","updated":"2024-10-10 08:48:25.000000000","message":"I don\u0027t think this should be necessary. I already fixed this in https://review.opendev.org/c/openstack/openstackdocstheme/+/927486 but it\u0027s not been released yet as I\u0027m waiting on https://review.opendev.org/c/openstack/releases/+/927529 to merge.\n\nIs https://bugs.launchpad.net/ironic/+bug/2078361 still reproducible with `master` of openstackdocstheme?","commit_id":"360ae5d4b6078755f94a3bb1592a82ceab65003c"},{"author":{"_account_id":35929,"name":"Mahnoor Asghar","display_name":"Mahnoor Asghar","email":"masghar@redhat.com","username":"mahnoorasghar"},"change_message_id":"7692fd7598e667f5dfa41c55b3b8164b6e6e32d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0e2f9047_cd6c8ad5","updated":"2024-09-27 16:34:14.000000000","message":"LGTM","commit_id":"360ae5d4b6078755f94a3bb1592a82ceab65003c"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"55d20e63d1881c5a06b35ea0f88b7d47ef92cfc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fd47cc4a_7086572f","updated":"2024-10-10 07:43:08.000000000","message":"The patch works but it can be maybe good to have more detail in the commit message.","commit_id":"360ae5d4b6078755f94a3bb1592a82ceab65003c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0176710eea2191c92da5815834bcbccb5ff542ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1ece88d9_dccb2c3c","updated":"2024-10-09 17:36:01.000000000","message":"lgtm","commit_id":"360ae5d4b6078755f94a3bb1592a82ceab65003c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"bf4649d104da6a8e1d03876dec883a025065f037","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"90820427_b5ba7f20","in_reply_to":"7e7d5dcf_1a416dc0","updated":"2024-10-10 14:06:50.000000000","message":"(It was my bad that that change dropped off from my memory...)\n\nSo I run doc build in a new venv with the current master of openstackdocstheme installed, and confirmed that we no longer see that warning.","commit_id":"360ae5d4b6078755f94a3bb1592a82ceab65003c"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"ff3de77f0823d5243f54b59ae5ba5716ae79c7eb","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e903f6e4_4e3fcd8a","in_reply_to":"90820427_b5ba7f20","updated":"2024-10-10 14:47:13.000000000","message":"I will be dropping this change now. Also, I just updated the bug on Launchpad.","commit_id":"360ae5d4b6078755f94a3bb1592a82ceab65003c"}]}