)]}'
{"/COMMIT_MSG":[{"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":"0defaa83bd9476450534b09bdf92ff8280733731","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Pass package list to Ansible package modules"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ansible no longer squashes loops for the package modules, so packages"},{"line_number":10,"context_line":"are installed individually."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"TrivialFix"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_cf23ae57","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":3},"updated":"2019-10-26 13:36:27.000000000","message":"were","commit_id":"906426470157e622532337d4c373d87f82963b85"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"0ddac79e055dbfc291827c4e3f8f8139f6eb03f0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Pass package list to Ansible package modules"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ansible no longer squashes loops for the package modules, so packages"},{"line_number":10,"context_line":"are installed individually."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"TrivialFix"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_3ca27eaf","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":3},"in_reply_to":"3fa7e38b_cf23ae57","updated":"2019-10-31 11:25:59.000000000","message":"I normally write it in present tense, describing current behaviour, and future tense with the patch applied.","commit_id":"906426470157e622532337d4c373d87f82963b85"}],"ansible/roles/disable-selinux/tasks/main.yml":[{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"4de224c5c9fbbdc5831e068d061ad2506e84029f","unresolved":false,"context_lines":[{"line_number":2,"context_line":"- name: Ensure required packages are installed"},{"line_number":3,"context_line":"  yum:"},{"line_number":4,"context_line":"    name:"},{"line_number":5,"context_line":"      - libselinux-python"},{"line_number":6,"context_line":"    state: installed"},{"line_number":7,"context_line":"  become: True"},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_80e7bc0f","line":5,"updated":"2019-12-06 13:55:10.000000000","message":"nit: single item list is strange when you can use a string","commit_id":"906426470157e622532337d4c373d87f82963b85"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"8ca45c5b302dac8c410b596171bd673b21bdea25","unresolved":false,"context_lines":[{"line_number":2,"context_line":"- name: Ensure required packages are installed"},{"line_number":3,"context_line":"  yum:"},{"line_number":4,"context_line":"    name:"},{"line_number":5,"context_line":"      - libselinux-python"},{"line_number":6,"context_line":"    state: installed"},{"line_number":7,"context_line":"  become: True"},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_9b396dd9","line":5,"in_reply_to":"3fa7e38b_80e7bc0f","updated":"2019-12-06 14:57:14.000000000","message":"Counter-nit: Since this is a task to install *required packages*, not libselinux-python specifically, keeping it as a list makes it easier to extend later.","commit_id":"906426470157e622532337d4c373d87f82963b85"}]}