)]}'
{"nodepool/elements/infra-package-needs/install.d/10-pip-packages":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"2ce6def0d687d32679a321407b78a3220de2ba60","unresolved":false,"context_lines":[{"line_number":21,"context_line":"set -e"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# TODO(fungi): switch jobs to use /usr/git-review-env/bin/git-review"},{"line_number":24,"context_line":"packages\u003d\u0027git-review \u0027"},{"line_number":25,"context_line":"packages+\u003d\u0027tox \u0027"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"for package in $packages ; do"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9ff52f3f_1f5e23e6","line":24,"updated":"2017-04-12 21:38:19.000000000","message":"We pin to 1.25.0 for puppet today, maybe we should do the same here?","commit_id":"3250aa5b8b6a82fd09bedeb6a6c7f591a67fd22c"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"3da26e9c3d6761909413a5d553e55d975e85b8a4","unresolved":false,"context_lines":[{"line_number":21,"context_line":"set -e"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# TODO(fungi): switch jobs to use /usr/git-review-env/bin/git-review"},{"line_number":24,"context_line":"packages\u003d\u0027git-review \u0027"},{"line_number":25,"context_line":"packages+\u003d\u0027tox \u0027"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"for package in $packages ; do"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9ff52f3f_7fa08fa6","line":24,"in_reply_to":"9ff52f3f_1f5e23e6","updated":"2017-04-12 21:45:01.000000000","message":"Woops, you\u0027re right","commit_id":"3250aa5b8b6a82fd09bedeb6a6c7f591a67fd22c"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"2ce6def0d687d32679a321407b78a3220de2ba60","unresolved":false,"context_lines":[{"line_number":25,"context_line":"packages+\u003d\u0027tox \u0027"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"for package in $packages ; do"},{"line_number":28,"context_line":"    pip install $package"},{"line_number":29,"context_line":"done"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9ff52f3f_1f13c30f","line":28,"updated":"2017-04-12 21:38:19.000000000","message":"we\u0027ll need sudo I think","commit_id":"3250aa5b8b6a82fd09bedeb6a6c7f591a67fd22c"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"3da26e9c3d6761909413a5d553e55d975e85b8a4","unresolved":false,"context_lines":[{"line_number":25,"context_line":"packages+\u003d\u0027tox \u0027"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"for package in $packages ; do"},{"line_number":28,"context_line":"    pip install $package"},{"line_number":29,"context_line":"done"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9ff52f3f_dfc79b45","line":28,"in_reply_to":"9ff52f3f_1f13c30f","updated":"2017-04-12 21:45:01.000000000","message":"I don\u0027t think so, it works fine for me without sudo","commit_id":"3250aa5b8b6a82fd09bedeb6a6c7f591a67fd22c"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"a1c9379c5ce04e5556537bee6e81e38be491d9fc","unresolved":false,"context_lines":[{"line_number":25,"context_line":"packages+\u003d\u0027tox \u0027"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"for package in $packages ; do"},{"line_number":28,"context_line":"    pip install $package"},{"line_number":29,"context_line":"done"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9ff52f3f_df3afb32","line":28,"in_reply_to":"9ff52f3f_dfc79b45","updated":"2017-04-12 21:47:20.000000000","message":"Yup, my mistake.","commit_id":"3250aa5b8b6a82fd09bedeb6a6c7f591a67fd22c"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"748c27ae1ec48634d451e07e2dd86541f4ccf6e2","unresolved":false,"context_lines":[{"line_number":21,"context_line":"set -e"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# TODO(fungi): switch jobs to use /usr/git-review-env/bin/git-review"},{"line_number":24,"context_line":"packages\u003d\u0027git-review\u003d\u003di.25.0 \u0027"},{"line_number":25,"context_line":"packages+\u003d\u0027tox \u0027"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"for package in $packages ; do"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"9ff52f3f_d4765a3f","line":24,"range":{"start_line":24,"start_character":22,"end_line":24,"end_character":23},"updated":"2017-04-13 04:34:42.000000000","message":"That should be 1 (ONE) not i.","commit_id":"39ff9fec1dddb303dcffc1a2cda685d25b51f7fa"}]}