{"action":"created","issue":{"url":"https://api.github.com/repos/Tribler/tribler/issues/4122","repository_url":"https://api.github.com/repos/Tribler/tribler","labels_url":"https://api.github.com/repos/Tribler/tribler/issues/4122/labels{/name}","comments_url":"https://api.github.com/repos/Tribler/tribler/issues/4122/comments","events_url":"https://api.github.com/repos/Tribler/tribler/issues/4122/events","html_url":"https://github.com/Tribler/tribler/issues/4122","id":390087723,"node_id":"MDU6SXNzdWUzOTAwODc3MjM=","number":4122,"title":"Build: Can we eliminate pylint: disable=redefined-builtin for six.moves imports?","user":{"login":"cclauss","id":3709715,"node_id":"MDQ6VXNlcjM3MDk3MTU=","avatar_url":"https://avatars3.githubusercontent.com/u/3709715?v=4","gravatar_id":"","url":"https://api.github.com/users/cclauss","html_url":"https://github.com/cclauss","followers_url":"https://api.github.com/users/cclauss/followers","following_url":"https://api.github.com/users/cclauss/following{/other_user}","gists_url":"https://api.github.com/users/cclauss/gists{/gist_id}","starred_url":"https://api.github.com/users/cclauss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cclauss/subscriptions","organizations_url":"https://api.github.com/users/cclauss/orgs","repos_url":"https://api.github.com/users/cclauss/repos","events_url":"https://api.github.com/users/cclauss/events{/privacy}","received_events_url":"https://api.github.com/users/cclauss/received_events","type":"User","site_admin":false},"labels":[{"id":29695602,"node_id":"MDU6TGFiZWwyOTY5NTYwMg==","url":"https://api.github.com/repos/Tribler/tribler/labels/question","name":"question","color":"cc317c","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/Tribler/tribler/milestones/31","html_url":"https://github.com/Tribler/tribler/milestone/31","labels_url":"https://api.github.com/repos/Tribler/tribler/milestones/31/labels","id":3582233,"node_id":"MDk6TWlsZXN0b25lMzU4MjIzMw==","number":31,"title":"Python 3","description":"This milestone hosts all tickets related to our upcoming upgrade to Python 3.","creator":{"login":"devos50","id":1707075,"node_id":"MDQ6VXNlcjE3MDcwNzU=","avatar_url":"https://avatars3.githubusercontent.com/u/1707075?v=4","gravatar_id":"","url":"https://api.github.com/users/devos50","html_url":"https://github.com/devos50","followers_url":"https://api.github.com/users/devos50/followers","following_url":"https://api.github.com/users/devos50/following{/other_user}","gists_url":"https://api.github.com/users/devos50/gists{/gist_id}","starred_url":"https://api.github.com/users/devos50/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/devos50/subscriptions","organizations_url":"https://api.github.com/users/devos50/orgs","repos_url":"https://api.github.com/users/devos50/repos","events_url":"https://api.github.com/users/devos50/events{/privacy}","received_events_url":"https://api.github.com/users/devos50/received_events","type":"User","site_admin":false},"open_issues":4,"closed_issues":2,"state":"open","created_at":"2018-08-20T07:13:06Z","updated_at":"2018-12-12T16:31:41Z","due_on":null,"closed_at":null},"comments":3,"created_at":"2018-12-12T06:49:46Z","updated_at":"2018-12-12T17:17:18Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"__from six import unichr__  # pylint: disable=redefined-builtin\r\n\r\nThese pylint directives clutter the code.  We are ___purposefully___ using the six module to redefine certain builtins like __unichr__ that are incompatible with Python 3.  The default PyLint configurations allows __six.moves__ redefinition because it is such a common practice these daze.\r\n\r\nEven when we [modify our .pylintrc file to allow __six.moves__ redefinition](https://github.com/Tribler/tribler/pull/4121/files#diff-d6d3d73911fd3ad6f39e4027f763b307R117) our PyLint _still_ flags these and forces us to clutter up our code with distracting comments.\r\n\r\nDoes the build PyLint command resets __--redefining-builtins-modules__ to have a more restrictive value than the PyLint default?\r\n"},"comment":{"url":"https://api.github.com/repos/Tribler/tribler/issues/comments/446668702","html_url":"https://github.com/Tribler/tribler/issues/4122#issuecomment-446668702","issue_url":"https://api.github.com/repos/Tribler/tribler/issues/4122","id":446668702,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0NjY2ODcwMg==","user":{"login":"qstokkink","id":3630389,"node_id":"MDQ6VXNlcjM2MzAzODk=","avatar_url":"https://avatars1.githubusercontent.com/u/3630389?v=4","gravatar_id":"","url":"https://api.github.com/users/qstokkink","html_url":"https://github.com/qstokkink","followers_url":"https://api.github.com/users/qstokkink/followers","following_url":"https://api.github.com/users/qstokkink/following{/other_user}","gists_url":"https://api.github.com/users/qstokkink/gists{/gist_id}","starred_url":"https://api.github.com/users/qstokkink/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/qstokkink/subscriptions","organizations_url":"https://api.github.com/users/qstokkink/orgs","repos_url":"https://api.github.com/users/qstokkink/repos","events_url":"https://api.github.com/users/qstokkink/events{/privacy}","received_events_url":"https://api.github.com/users/qstokkink/received_events","type":"User","site_admin":false},"created_at":"2018-12-12T17:17:18Z","updated_at":"2018-12-12T17:17:18Z","author_association":"CONTRIBUTOR","body":">  _exact_ PyLint command that is used\r\n\r\nhttps://github.com/Tribler/gumby/blob/devel/scripts/run_nosetests_for_jenkins.sh#L51"}}
{"actor":{"display_login":"qstokkink"}}