)]}'
{"roles/openafs-client/tasks/openafs-client/Debian.yaml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"bca2e619f49a61727bae7489684d50d167917e2d","unresolved":false,"context_lines":[{"line_number":19,"context_line":"  package:"},{"line_number":20,"context_line":"    name:"},{"line_number":21,"context_line":"      - openafs-modules-dkms"},{"line_number":22,"context_line":"    state: latest # noqa 403"},{"line_number":23,"context_line":"  become: yes"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Install client packages"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7faddb67_de65141f","line":22,"updated":"2019-08-08 09:45:14.000000000","message":"I don\u0027t like this. IMHO there are valid reasons to install latest packages and having to add some comment just to please lint will make things less readable and more tedious to work with.\n\nCan we exclude this instead?","commit_id":"ae8b6fd06385f17a2b2fdd9146f1413cf8f1e242"}],"tox.ini":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"d5ff39d3f4dc17a4f91e3d7eac50b662b1d18563","unresolved":false,"context_lines":[{"line_number":44,"context_line":"  #"},{"line_number":45,"context_line":"  # 403: Package installs should not use latest"},{"line_number":46,"context_line":"  #  We need in several places latest, ignore the rule and let"},{"line_number":47,"context_line":"  #  developers decide where to use it."},{"line_number":48,"context_line":"  bash -c \"find roles playbooks -type f -regex \u0027.*.y[a]?ml\u0027 -print0 | xargs -t -n1 -0 \\"},{"line_number":49,"context_line":"    ansible-lint -x ANSIBLE0004 -x ANSIBLE0006 -x ANSIBLE0007 -x ANSIBLE0010 \\"},{"line_number":50,"context_line":"                 -x ANSIBLE0011 -x ANSIBLE0012 -x ANSIBLE0013 -x ANSIBLE0015 \\"}],"source_content_type":"text/x-properties","patch_set":2,"id":"7faddb67_e7015106","line":47,"updated":"2019-08-08 23:40:17.000000000","message":"arrrghh, sorry to make this harder, but now I notice that ANSIBLE0010 should do the same thing.\n\nIt seems that at some point ansible-lint changed ANSIBLE0010 into a \"tag\" and replaced it with \"id\" 403 (where tags are generally something more generic like \"readability\" or \"formatting\" to select groups of tests)\n\n... but \"-x\" should be skipping tags, and error 403 should be tagged ANSIBLE0010, so *in theory* it should be skipped.\n\nI don\u0027t know if we want to bother debugging this in ansible-lint, but it seems like a bug :/  But here at least, I think we can remove the ANSIBLE0010 at least for clarity","commit_id":"58043311880dd58acef8f0cb7edd85cc5647ee21"}]}
