GitHub PullRequestReviewCommentEvent at 2014-01-12 09:41:30
Repository: python-fedora
Actor: abadger
Payload: {"comment":{"url":"https://api.github.com/repos/fedora-infra/python-fedora/pulls/comments/8813282","id":8813282,"diff_hunk":"@@ -32,16 +32,17 @@\n #   tg1utils.  We use a wildcard import just to import the names for functions\n #   into this namespace.\n # :W0614: Ditto.\n-from fedora.tg.tg1utils import * #pylint:disable-msg=W0614,W0401\n+from fedora.tg.tg1utils import *  # pylint:disable-msg=W0614,W0401\n \n import warnings\n \n warnings.warn('fedora.tg.util is deprecated.  Switch to one of these'\n-    ' instead:'\n-    '   TG1 apps: fedora.tg.utils'\n-    '   TG2 apps: fedora.tg2.utils.  This file will disappear in 0.4',\n-    DeprecationWarning, stacklevel=2)\n+              ' instead:'\n+              '   TG1 apps: fedora.tg.utils'\n+              '   TG2 apps: fedora.tg2.utils.  This file will disappear'\n+              '             in 0.4',\n+              DeprecationWarning, stacklevel=2)","path":"fedora/tg/util.py","position":null,"original_position":39,"commit_id":"96957a19e116283d83a35f3b0341f5a48a1a8ee2","original_commit_id":"f6d992731e02b07bf0592c98afa372298b0cd18f","user":{"login":"abadger","id":209242,"avatar_url":"https://gravatar.com/avatar/7a9c1d88f484c9806bceca0d6d91e948?d=https%3A%2F%2Fidenticons.github.com%2Faa03b4fbd250c9a07e45662b66fe71c3.png&r=x","gravatar_id":"7a9c1d88f484c9806bceca0d6d91e948","url":"https://api.github.com/users/abadger","html_url":"https://github.com/abadger","followers_url":"https://api.github.com/users/abadger/followers","following_url":"https://api.github.com/users/abadger/following{/other_user}","gists_url":"https://api.github.com/users/abadger/gists{/gist_id}","starred_url":"https://api.github.com/users/abadger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abadger/subscriptions","organizations_url":"https://api.github.com/users/abadger/orgs","repos_url":"https://api.github.com/users/abadger/repos","events_url":"https://api.github.com/users/abadger/events{/privacy}","received_events_url":"https://api.github.com/users/abadger/received_events","type":"User","site_admin":false},"body":"No, I think there's probably tools out in the wild that are expecting that Exceptions and their messages are printed on a single line.  Putting in newlines would break those tools.","created_at":"2014-01-12T09:41:30Z","updated_at":"2014-01-12T09:41:30Z","html_url":"https://github.com/fedora-infra/python-fedora/pull/71#discussion_r8813282","pull_request_url":"https://api.github.com/repos/fedora-infra/python-fedora/pulls/71","_links":{"self":{"href":"https://api.github.com/repos/fedora-infra/python-fedora/pulls/comments/8813282"},"html":{"href":"https://github.com/fedora-infra/python-fedora/pull/71#discussion_r8813282"},"pull_request":{"href":"https://api.github.com/repos/fedora-infra/python-fedora/pulls/71"}}}}