)]}'
{"tox.ini":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4078989eea5a3d153aeab28f87907fe7b6574a4e","unresolved":false,"context_lines":[{"line_number":172,"context_line":"# H105 don\u0027t use author tags. We use version control instead"},{"line_number":173,"context_line":"# H302 import only modules"},{"line_number":174,"context_line":"# H306 imports not in alphabetical order"},{"line_number":175,"context_line":"# H328 old style class declaration, use new style (inherit from `object`)"},{"line_number":176,"context_line":"# H402 one line docstring needs punctuation"},{"line_number":177,"context_line":"# H404 multi line docstring should start with a summary"},{"line_number":178,"context_line":"# H405 multi line docstring summary not separated with an empty line"}],"source_content_type":"text/x-properties","patch_set":3,"id":"5fc1f717_7efb1846","side":"PARENT","line":175,"updated":"2019-03-27 06:59:42.000000000","message":"This is a typo, the correct code is H238 and we do have four instances where we hit it. You may want to correct the ID and the ordering here and below, but not drop it.\n\nOr leave it as is and I\u0027ll spin up another patch to try and cut down our ignore list, H306 would be another good target IMO.","commit_id":"33ca0b87891228d86e440e929feeb324353b39d3"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"5fbf0798f6409ee086575d2fad48edbf811295c7","unresolved":false,"context_lines":[{"line_number":172,"context_line":"# H105 don\u0027t use author tags. We use version control instead"},{"line_number":173,"context_line":"# H302 import only modules"},{"line_number":174,"context_line":"# H306 imports not in alphabetical order"},{"line_number":175,"context_line":"# H328 old style class declaration, use new style (inherit from `object`)"},{"line_number":176,"context_line":"# H402 one line docstring needs punctuation"},{"line_number":177,"context_line":"# H404 multi line docstring should start with a summary"},{"line_number":178,"context_line":"# H405 multi line docstring summary not separated with an empty line"}],"source_content_type":"text/x-properties","patch_set":3,"id":"5fc1f717_3ddd89dc","side":"PARENT","line":175,"in_reply_to":"5fc1f717_7efb1846","updated":"2019-03-27 16:57:23.000000000","message":"Good catch - wouldn\u0027t mind seeing this list reduced if you have some time.","commit_id":"33ca0b87891228d86e440e929feeb324353b39d3"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4078989eea5a3d153aeab28f87907fe7b6574a4e","unresolved":false,"context_lines":[{"line_number":6,"context_line":"[testenv]"},{"line_number":7,"context_line":"setenv \u003d"},{"line_number":8,"context_line":"  VIRTUAL_ENV\u003d{envdir}"},{"line_number":9,"context_line":"  LANGUAGE\u003den_US"},{"line_number":10,"context_line":"  LC_ALL\u003den_US.utf-8"},{"line_number":11,"context_line":"  OS_LOG_CAPTURE\u003d{env:OS_LOG_CAPTURE:true}"},{"line_number":12,"context_line":"  OS_STDOUT_CAPTURE\u003d{env:OS_STDOUT_CAPTURE:true}"},{"line_number":13,"context_line":"  OS_STDERR_CAPTURE\u003d{env:OS_STDERR_CAPTURE:true}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"5fc1f717_be78a0c3","line":10,"range":{"start_line":9,"start_character":2,"end_line":10,"end_character":20},"updated":"2019-03-27 06:59:42.000000000","message":"I haven\u0027t tested that, but how about using the generic \"C\"/\"C.utf-8\" instead?","commit_id":"a503cb83cb79581e38f636295e94e95b88f9b029"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4078989eea5a3d153aeab28f87907fe7b6574a4e","unresolved":false,"context_lines":[{"line_number":29,"context_line":"  find . -type f -name \"*.py[c|o]\" -delete"},{"line_number":30,"context_line":"  rm -f .testrepository/times.dbm"},{"line_number":31,"context_line":"deps \u003d"},{"line_number":32,"context_line":"  -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt}"},{"line_number":33,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":34,"context_line":"  -r{toxinidir}/test-requirements.txt"},{"line_number":35,"context_line":"whitelist_externals \u003d"}],"source_content_type":"text/x-properties","patch_set":3,"id":"5fc1f717_7ea9d821","line":32,"updated":"2019-03-27 06:59:42.000000000","message":"I\u0027m not sure whether we can rely on the master branch being the default branch, better keep the explicit branch selector.","commit_id":"a503cb83cb79581e38f636295e94e95b88f9b029"}]}