)]}'
{"tox.ini":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"914613468dc2da1e23bee08d953b9861df968066","unresolved":true,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"[testenv:releasenotes]"},{"line_number":66,"context_line":"deps \u003d"},{"line_number":67,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":68,"context_line":"allowlist_externals \u003d"},{"line_number":69,"context_line":"  rm"},{"line_number":70,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"442e0149_5056228e","line":67,"range":{"start_line":67,"start_character":2,"end_line":67,"end_character":36},"updated":"2026-04-28 08:16:43.000000000","message":"I did not tested with this patch, but I think this is still loaded even without this statement, like in other repos.","commit_id":"bef50670a1d588ae7c0be67d41190b90b2d77b32"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"9248827cce9432192ff8ed8fcde86961c6b001cd","unresolved":true,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"[testenv:releasenotes]"},{"line_number":66,"context_line":"deps \u003d"},{"line_number":67,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":68,"context_line":"allowlist_externals \u003d"},{"line_number":69,"context_line":"  rm"},{"line_number":70,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"674f376e_5434cc40","line":67,"range":{"start_line":67,"start_character":2,"end_line":67,"end_character":36},"in_reply_to":"442e0149_5056228e","updated":"2026-04-29 12:41:34.000000000","message":"Assuming I understand your comment correctly, if we remove `deps` from here then we will fallback to the definition in `[testenv]` (line 11 above). That **does not** include `-r{toxinidir}/doc/requirements.txt` so we will not install these dependencies.\n\nI believe the reason @kajinamit@oss.nttdata.com is doing this is because we do not require non-doc dependencies for release notes, which is reasonable.","commit_id":"bef50670a1d588ae7c0be67d41190b90b2d77b32"}]}
