)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"215b0725697fa8de15f0cdf6eb4eea3ffe5cb6bc","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sorin Sbarnea \u003cssbarnea@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-11-14 12:34:45 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix pre-commit execution"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- assures pre-commit *fully* runs on CI"},{"line_number":10,"context_line":"- fixes broken newlines caused by incomplete CI execution"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"3fa7e38b_ba33b560","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":24},"updated":"2019-11-22 23:21:19.000000000","message":"pre-commit execution isn\u0027t broken, this is a stylistic change, which we should do; the commit title should be updated.\n\nWhile what is being done is in this commit message we shoudl explain why we\u0027re doing it.","commit_id":"6db1c91617434cd12ca2d0813698a62a06472b1d"}],"requirements.txt":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"59e2588a32bfe953a3808de2a64426875ff347dd","unresolved":false,"context_lines":[{"line_number":1,"context_line":"ansible"},{"line_number":2,"context_line":"pbr\u003e\u003d1.6"},{"line_number":3,"context_line":"paunch\u003e\u003d5.1.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":6,"id":"3fa7e38b_f6ed65f0","line":1,"updated":"2019-11-12 15:32:36.000000000","message":"sadly we cant have this requirement here. The requirements.txt file is managed by the requirements project, which ansible is not compatible.","commit_id":"aad5d7e28b427ba42def96390bdee55f8338735f"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"b82e5283cd2609941045cb6142ece8328aeb9d75","unresolved":false,"context_lines":[{"line_number":1,"context_line":"ansible"},{"line_number":2,"context_line":"pbr\u003e\u003d1.6"},{"line_number":3,"context_line":"paunch\u003e\u003d5.1.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":6,"id":"3fa7e38b_568d5933","line":1,"in_reply_to":"3fa7e38b_f6ed65f0","updated":"2019-11-12 15:34:13.000000000","message":"So mainly we \"lie\" about project requirements to keep it working? The reality is that the code itself cannot be used without ansible.","commit_id":"aad5d7e28b427ba42def96390bdee55f8338735f"}],"test-requirements.txt":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"215b0725697fa8de15f0cdf6eb4eea3ffe5cb6bc","unresolved":false,"context_lines":[{"line_number":1,"context_line":"ansible\u003e\u003d2.8  # Needed to install galaxy deps for both linters and molecule"},{"line_number":2,"context_line":"netaddr # BSD"}],"source_content_type":"text/plain","patch_set":10,"id":"3fa7e38b_fa084d91","line":1,"range":{"start_line":1,"start_character":7,"end_line":1,"end_character":12},"updated":"2019-11-22 23:21:19.000000000","message":"is the \u003e\u003d2.8 version required here if we not setting an upper bound?","commit_id":"6db1c91617434cd12ca2d0813698a62a06472b1d"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"215b0725697fa8de15f0cdf6eb4eea3ffe5cb6bc","unresolved":false,"context_lines":[{"line_number":1,"context_line":"ansible\u003e\u003d2.8  # Needed to install galaxy deps for both linters and molecule"},{"line_number":2,"context_line":"netaddr # BSD"}],"source_content_type":"text/plain","patch_set":10,"id":"3fa7e38b_1ad689f3","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":12},"updated":"2019-11-22 23:21:19.000000000","message":"sadly this requirement file is maintained by upstream requirements and upstream does not permit ansible as a requirement. This will need to remain in the molecule requirements file or in yet another requirement file if we really need to move it.","commit_id":"6db1c91617434cd12ca2d0813698a62a06472b1d"}],"tox.ini":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"215b0725697fa8de15f0cdf6eb4eea3ffe5cb6bc","unresolved":false,"context_lines":[{"line_number":51,"context_line":"deps \u003d"},{"line_number":52,"context_line":"  -r {toxinidir}/requirements.txt"},{"line_number":53,"context_line":"  -r {toxinidir}/test-requirements.txt  # ansible-galaxy"},{"line_number":54,"context_line":"  pre-commit\u003e\u003d1.20"},{"line_number":55,"context_line":"commands \u003d"},{"line_number":56,"context_line":"  {[testenv]commands}"},{"line_number":57,"context_line":"  python -m pre_commit run {posargs:} -a"}],"source_content_type":"text/x-properties","patch_set":10,"id":"3fa7e38b_bae5f5dc","line":54,"range":{"start_line":54,"start_character":2,"end_line":54,"end_character":18},"updated":"2019-11-22 23:21:19.000000000","message":"this dep should remain in the test-requirements.","commit_id":"6db1c91617434cd12ca2d0813698a62a06472b1d"}]}