{"action":"created","issue":{"url":"https://api.github.com/repos/google/yapf/issues/72","labels_url":"https://api.github.com/repos/google/yapf/issues/72/labels{/name}","comments_url":"https://api.github.com/repos/google/yapf/issues/72/comments","events_url":"https://api.github.com/repos/google/yapf/issues/72/events","html_url":"https://github.com/google/yapf/pull/72","id":66674760,"number":72,"title":"Improve handling of filenames on the command line","user":{"login":"sbc100","id":515813,"avatar_url":"https://avatars.githubusercontent.com/u/515813?v=3","gravatar_id":"","url":"https://api.github.com/users/sbc100","html_url":"https://github.com/sbc100","followers_url":"https://api.github.com/users/sbc100/followers","following_url":"https://api.github.com/users/sbc100/following{/other_user}","gists_url":"https://api.github.com/users/sbc100/gists{/gist_id}","starred_url":"https://api.github.com/users/sbc100/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbc100/subscriptions","organizations_url":"https://api.github.com/users/sbc100/orgs","repos_url":"https://api.github.com/users/sbc100/repos","events_url":"https://api.github.com/users/sbc100/events{/privacy}","received_events_url":"https://api.github.com/users/sbc100/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/google/yapf/labels/cla:%20no","name":"cla: no","color":"e11d21"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-04-06T18:35:20Z","updated_at":"2015-04-07T18:37:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/google/yapf/pulls/72","html_url":"https://github.com/google/yapf/pull/72","diff_url":"https://github.com/google/yapf/pull/72.diff","patch_url":"https://github.com/google/yapf/pull/72.patch"},"body":"Don't call GetCommandLineFiles unless there are positional\r\narguments (args.files), and only pass the positional parameters\r\n(otherwise GetCommandLineFiles ends up processing all the args\r\nas well).\r\n\r\nPrior to this change if you run 'yapf -i foo.c' it will give you\r\nthe unhelpful error:\r\n'cannot use --in_place or --diff flags when reading from stdin'\r\n\r\nAdd a unit test for this case. Would add here but would prefer\r\nto use 'mock' and don't see a dependency on 'mock' yet.\r\n\r\nAlso, give a useful error message when positional paramters\r\nare specified but not arguments were given."},"comment":{"url":"https://api.github.com/repos/google/yapf/issues/comments/90693232","html_url":"https://github.com/google/yapf/pull/72#issuecomment-90693232","issue_url":"https://api.github.com/repos/google/yapf/issues/72","id":90693232,"user":{"login":"googlebot","id":5384588,"avatar_url":"https://avatars.githubusercontent.com/u/5384588?v=3","gravatar_id":"","url":"https://api.github.com/users/googlebot","html_url":"https://github.com/googlebot","followers_url":"https://api.github.com/users/googlebot/followers","following_url":"https://api.github.com/users/googlebot/following{/other_user}","gists_url":"https://api.github.com/users/googlebot/gists{/gist_id}","starred_url":"https://api.github.com/users/googlebot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/googlebot/subscriptions","organizations_url":"https://api.github.com/users/googlebot/orgs","repos_url":"https://api.github.com/users/googlebot/repos","events_url":"https://api.github.com/users/googlebot/events{/privacy}","received_events_url":"https://api.github.com/users/googlebot/received_events","type":"User","site_admin":false},"created_at":"2015-04-07T18:37:45Z","updated_at":"2015-04-07T18:37:45Z","body":"CLAs look good, thanks!\n\n<!-- ok -->"}}