)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":19779,"name":"Chason Chan","display_name":"chenxing","email":"chason.chan@foxmail.com","username":"chenxing"},"change_message_id":"da0b8b440110def04be04e3dd1e1980029bfe293","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This makes tox actions against docs better suited for iterative"},{"line_number":10,"context_line":"work (rise-rinse-repeat)."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- doc envs do not require k-a to be (re)installed (skip_install)"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- this change preserves doctrees and builds between runs (no rm,"},{"line_number":15,"context_line":"  no -a, no -E); this can be overridden e.g. like this:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"180568f0_38b08878","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":64},"updated":"2021-02-02 04:08:08.000000000","message":"I approve to this change. 👍","commit_id":"2227094f79d1705cdcf59a42cd7df8e5280d531a"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"0968daa3bde59505c9294e1e67099f516d5d170f","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- doc envs do not require k-a to be (re)installed (skip_install)"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- this change preserves doctrees and builds between runs (no rm,"},{"line_number":15,"context_line":"  no -a, no -E); this can be overridden e.g. like this:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    tox -e docs -- -E"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"  this does not apply to releasenotes which never refresh without"},{"line_number":20,"context_line":"  -E, so this change keeps -E for releasenotes (-E implies -a);"},{"line_number":21,"context_line":"  this contributes the majority of performance gain for iterative"},{"line_number":22,"context_line":"  work"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- this change allows extra posargs to be passed to these commands"},{"line_number":25,"context_line":"  (hence the above works)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"cc000b5e_2a87fd34","line":22,"range":{"start_line":14,"start_character":0,"end_line":22,"end_character":6},"updated":"2020-12-15 17:30:51.000000000","message":"This is fine for iterating, but what happens if you change to a different commit? The docs build time is longer than I\u0027d like, but I\u0027d rather ensure it is correct than fast.","commit_id":"2227094f79d1705cdcf59a42cd7df8e5280d531a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"83b0ba5b8d103d82f4dc52a33101599a328e064d","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- doc envs do not require k-a to be (re)installed (skip_install)"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- this change preserves doctrees and builds between runs (no rm,"},{"line_number":15,"context_line":"  no -a, no -E); this can be overridden e.g. like this:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    tox -e docs -- -E"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"  this does not apply to releasenotes which never refresh without"},{"line_number":20,"context_line":"  -E, so this change keeps -E for releasenotes (-E implies -a);"},{"line_number":21,"context_line":"  this contributes the majority of performance gain for iterative"},{"line_number":22,"context_line":"  work"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- this change allows extra posargs to be passed to these commands"},{"line_number":25,"context_line":"  (hence the above works)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"feb430e8_f3e10bd8","line":22,"range":{"start_line":14,"start_character":0,"end_line":22,"end_character":6},"in_reply_to":"455aa74d_277a07e2","updated":"2020-12-15 19:51:19.000000000","message":"I mean, you can switch your branch and do `tox -e docs -- -E` if it has issues. Try yourself. I find this compromise a better one.","commit_id":"2227094f79d1705cdcf59a42cd7df8e5280d531a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"81e7719358c412dcd39bcbe2aa091654791e4331","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- doc envs do not require k-a to be (re)installed (skip_install)"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- this change preserves doctrees and builds between runs (no rm,"},{"line_number":15,"context_line":"  no -a, no -E); this can be overridden e.g. like this:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    tox -e docs -- -E"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"  this does not apply to releasenotes which never refresh without"},{"line_number":20,"context_line":"  -E, so this change keeps -E for releasenotes (-E implies -a);"},{"line_number":21,"context_line":"  this contributes the majority of performance gain for iterative"},{"line_number":22,"context_line":"  work"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- this change allows extra posargs to be passed to these commands"},{"line_number":25,"context_line":"  (hence the above works)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ef517a82_947a9b95","line":22,"range":{"start_line":14,"start_character":0,"end_line":22,"end_character":6},"in_reply_to":"cc000b5e_2a87fd34","updated":"2020-12-15 18:33:03.000000000","message":"Well, I find it hard to miss it being wrong. Also, as the commit message says - it is easy to get the old behaviour.","commit_id":"2227094f79d1705cdcf59a42cd7df8e5280d531a"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"ef20acb2571c1a8b33aff7b3e5b6e1b356ff3900","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- doc envs do not require k-a to be (re)installed (skip_install)"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- this change preserves doctrees and builds between runs (no rm,"},{"line_number":15,"context_line":"  no -a, no -E); this can be overridden e.g. like this:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    tox -e docs -- -E"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"  this does not apply to releasenotes which never refresh without"},{"line_number":20,"context_line":"  -E, so this change keeps -E for releasenotes (-E implies -a);"},{"line_number":21,"context_line":"  this contributes the majority of performance gain for iterative"},{"line_number":22,"context_line":"  work"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- this change allows extra posargs to be passed to these commands"},{"line_number":25,"context_line":"  (hence the above works)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"455aa74d_277a07e2","line":22,"range":{"start_line":14,"start_character":0,"end_line":22,"end_character":6},"in_reply_to":"ef517a82_947a9b95","updated":"2020-12-15 19:46:58.000000000","message":"What do you mean? Surely it is more likely to be wrong if we don\u0027t discard and rebuild.","commit_id":"2227094f79d1705cdcf59a42cd7df8e5280d531a"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"735e3841906292071d42959fc209511d8889acb3","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- doc envs do not require k-a to be (re)installed (skip_install)"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- this change preserves doctrees and builds between runs (no rm,"},{"line_number":15,"context_line":"  no -a, no -E); this can be overridden e.g. like this:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    tox -e docs -- -E"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"  this does not apply to releasenotes which never refresh without"},{"line_number":20,"context_line":"  -E, so this change keeps -E for releasenotes (-E implies -a);"},{"line_number":21,"context_line":"  this contributes the majority of performance gain for iterative"},{"line_number":22,"context_line":"  work"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- this change allows extra posargs to be passed to these commands"},{"line_number":25,"context_line":"  (hence the above works)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a003377_e4387249","line":22,"range":{"start_line":14,"start_character":0,"end_line":22,"end_character":6},"in_reply_to":"feb430e8_f3e10bd8","updated":"2020-12-17 10:12:03.000000000","message":"Only if you know to do that. I won\u0027t block this but I\u0027m not approving.","commit_id":"2227094f79d1705cdcf59a42cd7df8e5280d531a"}]}