)]}'
{"tools/jenkins-projects-checks.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"31e7f2111ccdc9de96eff96ae5857ec0884a57c8","unresolved":false,"context_lines":[{"line_number":187,"context_line":"        if \u0027builders\u0027 in entry:"},{"line_number":188,"context_line":"            # tox is in an orderdict, so we need iterate manually"},{"line_number":189,"context_line":"            for b in entry[\u0027builders\u0027]:"},{"line_number":190,"context_line":"                if isinstance(b, dict):"},{"line_number":191,"context_line":"                    if \u0027tox\u0027 in b:"},{"line_number":192,"context_line":"                        print(\"ERROR: Use \u0027run-tox\u0027 instead of \u0027tox\u0027 \""},{"line_number":193,"context_line":"                              \"builder in \u0027%s\u0027:\" % entry[\u0027name\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_1c110b53","line":190,"updated":"2016-11-15 17:07:36.000000000","message":"I think you also need to include:\n\n  else if isinstance(b, str) and b \u003d\u003d \u0027tox\u0027:\n      # set as error.\n\nBecause the JJB tox builder can run without any parameters so may be used as a string and not a dict. See http://docs.openstack.org/infra/jenkins-job-builder/builders.html#builders.tox","commit_id":"aab0cb3e747bb2dd6eee4d7d5f2ad74dbc91dd00"}]}