)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"a30a10a86a90025536c5d5ac13c483089171d9fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"87bcfe0c_901f3595","updated":"2024-03-28 12:44:51.000000000","message":"Thank you for your review Dmitry. Uploaded a revised version.","commit_id":"a54739ecc60daa4ec16b2b6ba9bbee69a6b955f2"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"0b563d27cf6c957f18962ea9974189559dfc8cb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6eecb338_5d02deed","updated":"2024-03-28 21:33:11.000000000","message":"recheck\n\npassed test locally and passed CI in previous run","commit_id":"4a1e6f241e0653e85f5543918c8c8985b21d1851"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"11036c7a123f49832486a49399754182997091e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"61a77720_312f7fee","updated":"2024-04-02 02:24:35.000000000","message":"Looks like there was a missing newline between the imports in manager/job_collection.py which got missed because of other pre-existing pep8 issues (which are being addressed in another in-review patch).","commit_id":"d8dc61bc1cdbd867786ac6fd7150c5f50e8620b9"}],"sushy_oem_idrac/resources/manager/job_collection.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ff24d23a8e4481e3a3218e524ca84fa5c004bf78","unresolved":true,"context_lines":[{"line_number":49,"context_line":"        job_response \u003d self._conn.get(job_expand_uri)"},{"line_number":50,"context_line":"        data \u003d job_response.json()"},{"line_number":51,"context_line":"        for job in data[u\u0027Members\u0027]:"},{"line_number":52,"context_line":"            if ((job[u\u0027JobState\u0027] \u003d\u003d \u0027Scheduled\u0027) or ("},{"line_number":53,"context_line":"                    job[u\u0027JobState\u0027] \u003d\u003d \u0027Running\u0027) or ("},{"line_number":54,"context_line":"                    job[u\u0027JobState\u0027] \u003d\u003d \u0027Paused\u0027)):"},{"line_number":55,"context_line":"                unfinished_jobs.append(job[\u0027Id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"ca25c965_d9e3518b","line":52,"updated":"2024-03-28 09:56:35.000000000","message":"Could you move the acceptable states to a constant and use \"in\" operation?","commit_id":"a54739ecc60daa4ec16b2b6ba9bbee69a6b955f2"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"a30a10a86a90025536c5d5ac13c483089171d9fc","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        job_response \u003d self._conn.get(job_expand_uri)"},{"line_number":50,"context_line":"        data \u003d job_response.json()"},{"line_number":51,"context_line":"        for job in data[u\u0027Members\u0027]:"},{"line_number":52,"context_line":"            if ((job[u\u0027JobState\u0027] \u003d\u003d \u0027Scheduled\u0027) or ("},{"line_number":53,"context_line":"                    job[u\u0027JobState\u0027] \u003d\u003d \u0027Running\u0027) or ("},{"line_number":54,"context_line":"                    job[u\u0027JobState\u0027] \u003d\u003d \u0027Paused\u0027)):"},{"line_number":55,"context_line":"                unfinished_jobs.append(job[\u0027Id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"20ab9d2b_ff58527f","line":52,"in_reply_to":"ca25c965_d9e3518b","updated":"2024-03-28 12:44:51.000000000","message":"Thank you for your suggestion. Done.","commit_id":"a54739ecc60daa4ec16b2b6ba9bbee69a6b955f2"}]}