)]}'
{"ansible-lint/YAMLdictchecker.py":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"7690fa908d1701ab51569773c1ce7f9aa8712a97","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from ansiblelint import AnsibleLintRule"},{"line_number":19,"context_line":"class YAMLdictchecker(AnsibleLintRule):"},{"line_number":20,"context_line":"    id \u003d \u0027OSA0001\u0027"},{"line_number":21,"context_line":"    shortdesc \u003d \u0027Use \":\" YAML format\u0027"},{"line_number":22,"context_line":"    description \u003d \u0027\u0027"},{"line_number":23,"context_line":"    tags \u003d [\u0027task\u0027]"},{"line_number":24,"context_line":"    def match(self, file,line):"},{"line_number":25,"context_line":"        for l in line.split(\" \"):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_4155e8f7","line":22,"range":{"start_line":21,"start_character":0,"end_line":22,"end_character":20},"updated":"2016-10-07 17:11:57.000000000","message":"Ideally this should be more descriptive. It should describe the problem and point to a URL where more details about what the right thing to do is. This patch should also include the documentation that gets pointed to.","commit_id":"a0212cf54777f25f8c30334ea30e3effbf5f3a6c"},{"author":{"_account_id":22778,"name":"Nish Patwa(nishpatwa007)","email":"nish.patwa@intel.com","username":"npatwa"},"change_message_id":"baa0b988c502eb90260f07bbc4ea10106dd1be33","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from ansiblelint import AnsibleLintRule"},{"line_number":19,"context_line":"class YAMLdictchecker(AnsibleLintRule):"},{"line_number":20,"context_line":"    id \u003d \u0027OSA0001\u0027"},{"line_number":21,"context_line":"    shortdesc \u003d \u0027Use \":\" YAML format\u0027"},{"line_number":22,"context_line":"    description \u003d \u0027\u0027"},{"line_number":23,"context_line":"    tags \u003d [\u0027task\u0027]"},{"line_number":24,"context_line":"    def match(self, file,line):"},{"line_number":25,"context_line":"        for l in line.split(\" \"):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_f62790b3","line":22,"range":{"start_line":21,"start_character":0,"end_line":22,"end_character":20},"in_reply_to":"1a95cdbc_4155e8f7","updated":"2016-10-07 20:35:07.000000000","message":"Thanks for the review. I added a short description in my patch set and a link in the description. Let me know if anything needs to be added","commit_id":"a0212cf54777f25f8c30334ea30e3effbf5f3a6c"},{"author":{"_account_id":12892,"name":"Nolan Brubaker","email":"nolan.brubaker@rackspace.com","username":"nrb"},"change_message_id":"53a67b3524743abe80ca2c08c6dbe8c76bcfe6d1","unresolved":false,"context_lines":[{"line_number":25,"context_line":"        for l in line.split(\" \"):"},{"line_number":26,"context_line":"            if \"\u003d\" in l:"},{"line_number":27,"context_line":"                return True"},{"line_number":28,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_bd188ed4","line":28,"updated":"2016-10-07 17:02:51.000000000","message":"This will always return true - is that intentional?","commit_id":"a0212cf54777f25f8c30334ea30e3effbf5f3a6c"},{"author":{"_account_id":22778,"name":"Nish Patwa(nishpatwa007)","email":"nish.patwa@intel.com","username":"npatwa"},"change_message_id":"baa0b988c502eb90260f07bbc4ea10106dd1be33","unresolved":false,"context_lines":[{"line_number":25,"context_line":"        for l in line.split(\" \"):"},{"line_number":26,"context_line":"            if \"\u003d\" in l:"},{"line_number":27,"context_line":"                return True"},{"line_number":28,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_bb041118","line":28,"in_reply_to":"1a95cdbc_bd188ed4","updated":"2016-10-07 20:35:07.000000000","message":"Thanks for pointing out. I did it intentionally for testing but then I forgot to change it. It should be false and I uploaded a patch.","commit_id":"a0212cf54777f25f8c30334ea30e3effbf5f3a6c"},{"author":{"_account_id":22778,"name":"Nish Patwa(nishpatwa007)","email":"nish.patwa@intel.com","username":"npatwa"},"change_message_id":"5a09426eebff84676382e65ceedb023d7bdcce3c","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from ansiblelint import AnsibleLintRule"},{"line_number":20,"context_line":"class YAMLdictchecker(AnsibleLintRule):"},{"line_number":21,"context_line":"    id \u003d \u0027OSA0001\u0027"},{"line_number":22,"context_line":"    shortdesc \u003d \u0027Please use \":\" YAML dictionary format instead of \"\u003d\"\u0027"},{"line_number":23,"context_line":"    description \u003d \u0027Pleae follow YAML dicitionary format while creating\u0027"},{"line_number":24,"context_line":"    \u0027task and other roles in Ansible\u0027"},{"line_number":25,"context_line":"    \u0027Follow this url for examples of how to use YAML dictionary \u0027"},{"line_number":26,"context_line":"    \u0027format. \"http://docs.openstack.org/developer/openstack-ansible/\u0027"},{"line_number":27,"context_line":"    \u0027developer-docs/contribute.html#ansible-style-guide\"\u0027"},{"line_number":28,"context_line":"    tags \u003d [\u0027task\u0027]"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    def match(self, file, line):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_f63870a2","line":27,"range":{"start_line":22,"start_character":4,"end_line":27,"end_character":57},"updated":"2016-10-07 20:30:39.000000000","message":"Thanks for the review. I added a short description in my patch set and a link in the description. Let me know if anything needs to be added","commit_id":"cb0e68d42c9d119ec503eac9fdb4e51ecc3d6fb0"},{"author":{"_account_id":22778,"name":"Nish Patwa(nishpatwa007)","email":"nish.patwa@intel.com","username":"npatwa"},"change_message_id":"5a09426eebff84676382e65ceedb023d7bdcce3c","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        for l in line.split(\" \"):"},{"line_number":32,"context_line":"            if \"\u003d\" in l:"},{"line_number":33,"context_line":"                return True"},{"line_number":34,"context_line":"            return False"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_163ebc8b","line":34,"range":{"start_line":34,"start_character":19,"end_line":34,"end_character":24},"updated":"2016-10-07 20:30:39.000000000","message":"Thanks for pointing out. I did it intentionally for testing but then I forgot to change it. It should be false and I uploaded a patch.","commit_id":"cb0e68d42c9d119ec503eac9fdb4e51ecc3d6fb0"}]}