)]}'
{"id":"openstack%2Fpython-swiftclient~78772","triplet_id":"openstack%2Fpython-swiftclient~master~Ieb8acc8277e64633d29c5c77b2afc0c237cb4d2a","project":"openstack/python-swiftclient","branch":"master","topic":"bug/1288988","hashtags":[],"change_id":"Ieb8acc8277e64633d29c5c77b2afc0c237cb4d2a","subject":"Remove ignore of H501 hacking check","status":"ABANDONED","created":"2014-03-06 21:25:16.000000000","updated":"2014-03-07 05:48:59.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"b7712ff69e0283e6c97dddd0d6c4aa43011f1f7d","_number":78772,"virtual_id_number":78772,"owner":{"_account_id":6601,"name":"James Carey","email":"jecarey@yerac.com","username":"jecarey"},"actions":{},"labels":{"Verified":{"all":[{"date":"2014-03-06 21:25:16.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"rejected":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"disliked":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-2,"date":"2014-03-06 21:27:47.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},{"value":-1,"date":"2014-03-06 21:26:59.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2014-03-06 21:27:47.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},{"value":0,"date":"2014-03-06 21:26:59.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-03-06 21:25:16.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2014-03-06 21:26:59.000000000","updated_by":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"reviewer":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"state":"REVIEWER"},{"updated":"2014-03-06 21:27:47.000000000","updated_by":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"reviewer":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"state":"REVIEWER"}],"messages":[{"id":"59afbfda08b661e26c6b65b7ecbb2ea560c567f8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-06 21:25:16.000000000","message":"Patch Set 1:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"d386e324c16cb093f7767edfaa48bf6fb0ca3526","author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"date":"2014-03-06 21:26:59.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nisn\u0027t our tox.ini a whitelist not a blacklist?","accounts_in_message":[],"_revision_number":1},{"id":"4c2a5fbfc0831fc2a26a07c50cd3dcc2cb6a5539","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2014-03-06 21:27:47.000000000","message":"Patch Set 1: Do not merge\n\nthe select line in the config is enforcing those checks, not ignoring them","accounts_in_message":[],"_revision_number":1},{"id":"ecea2b95a2afa4dd54a54c17c97ed6a6496d37fc","author":{"_account_id":6601,"name":"James Carey","email":"jecarey@yerac.com","username":"jecarey"},"date":"2014-03-06 21:27:54.000000000","message":"Patch Set 1: Abandoned\n\nMisunderstood that it was ignoring the other checks.","accounts_in_message":[],"_revision_number":1},{"id":"c18965fde10454c430e008c5d941a06b1837cff0","author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"date":"2014-03-07 05:48:59.000000000","message":"Patch Set 1:\n\nwell fwiw I don\u0027t think it\u0027s *actually* enforcing H501 because of they\u0027re comma-space separated instead of just comma separated.  Plus flake8 by itself w/o hacking installed seems to assume that the select list is exhaustive and is ignoring the F,E,W checks with this tox.ini by default.\n\nSee https://review.openstack.org/#/c/77072/\n\nMy point was just that removing H501 certainly isn\u0027t going to enforce it.  But dropping the spaces might!","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"d878af609942a4960a8d72d02337f44e2fb462a2","revisions":{"d878af609942a4960a8d72d02337f44e2fb462a2":{"kind":"REWORK","_number":1,"created":"2014-03-06 21:25:16.000000000","uploader":{"_account_id":6601,"name":"James Carey","email":"jecarey@yerac.com","username":"jecarey"},"ref":"refs/changes/72/78772/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-swiftclient","ref":"refs/changes/72/78772/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-swiftclient refs/changes/72/78772/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-swiftclient refs/changes/72/78772/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-swiftclient refs/changes/72/78772/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-swiftclient refs/changes/72/78772/1"}}},"commit":{"parents":[{"commit":"d2f3563cde2e7dec28e1ea13c4b02b5063a4c916","subject":"Merge \"Help string format persistent\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-swiftclient/commit/d2f3563cde2e7dec28e1ea13c4b02b5063a4c916"}]}],"author":{"name":"James Carey","email":"jecarey@us.ibm.com","date":"2014-03-06 21:07:30.000000000","tz":0},"committer":{"name":"James Carey","email":"jecarey@us.ibm.com","date":"2014-03-06 21:07:30.000000000","tz":0},"subject":"Remove ignore of H501 hacking check","message":"Remove ignore of H501 hacking check\n\nThis check is ignored even though there aren\u0027t any uses of\nlocals().\n\nChange-Id: Ieb8acc8277e64633d29c5c77b2afc0c237cb4d2a\nCloses-Bug: 1288988\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-swiftclient/commit/d878af609942a4960a8d72d02337f44e2fb462a2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-swiftclient/commit/d878af609942a4960a8d72d02337f44e2fb462a2"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}