{"action":"created","issue":{"url":"https://api.github.com/repos/bramp/ffmpeg-cli-wrapper/issues/64","repository_url":"https://api.github.com/repos/bramp/ffmpeg-cli-wrapper","labels_url":"https://api.github.com/repos/bramp/ffmpeg-cli-wrapper/issues/64/labels{/name}","comments_url":"https://api.github.com/repos/bramp/ffmpeg-cli-wrapper/issues/64/comments","events_url":"https://api.github.com/repos/bramp/ffmpeg-cli-wrapper/issues/64/events","html_url":"https://github.com/bramp/ffmpeg-cli-wrapper/issues/64","id":174838189,"number":64,"title":"FFmpegBuilder API is not intuitive","user":{"login":"henrydev10","id":12461722,"avatar_url":"https://avatars.githubusercontent.com/u/12461722?v=3","gravatar_id":"","url":"https://api.github.com/users/henrydev10","html_url":"https://github.com/henrydev10","followers_url":"https://api.github.com/users/henrydev10/followers","following_url":"https://api.github.com/users/henrydev10/following{/other_user}","gists_url":"https://api.github.com/users/henrydev10/gists{/gist_id}","starred_url":"https://api.github.com/users/henrydev10/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/henrydev10/subscriptions","organizations_url":"https://api.github.com/users/henrydev10/orgs","repos_url":"https://api.github.com/users/henrydev10/repos","events_url":"https://api.github.com/users/henrydev10/events{/privacy}","received_events_url":"https://api.github.com/users/henrydev10/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/bramp/ffmpeg-cli-wrapper/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-09-02T20:07:26Z","updated_at":"2016-10-06T20:55:36Z","closed_at":"2016-10-06T05:11:36Z","body":"It seems that after `addOutput(\"filename\")` another builder is returned - `FFmpegOutputBuilder`.\r\n\r\nIn my opinion the `FFmpegOutputBuilder` should be self contained builder, and `FFmpegBuilder` should have a method `public FFmpegBuilder addOutputBuilder(FFmpegOutputBuilder outputBuilder)`.\r\n\r\nThis will be much clearer, builder which returns itself and not another builder. Since once another builder is returned, there is no going back to the `FFmpegBuilder`.\r\n\r\nThis will also simplify multiple output creations on the consumer side, if there are repetitions with slight variations of the output, e.g. video format.\r\n"},"comment":{"url":"https://api.github.com/repos/bramp/ffmpeg-cli-wrapper/issues/comments/252085899","html_url":"https://github.com/bramp/ffmpeg-cli-wrapper/issues/64#issuecomment-252085899","issue_url":"https://api.github.com/repos/bramp/ffmpeg-cli-wrapper/issues/64","id":252085899,"user":{"login":"henrydev10","id":12461722,"avatar_url":"https://avatars.githubusercontent.com/u/12461722?v=3","gravatar_id":"","url":"https://api.github.com/users/henrydev10","html_url":"https://github.com/henrydev10","followers_url":"https://api.github.com/users/henrydev10/followers","following_url":"https://api.github.com/users/henrydev10/following{/other_user}","gists_url":"https://api.github.com/users/henrydev10/gists{/gist_id}","starred_url":"https://api.github.com/users/henrydev10/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/henrydev10/subscriptions","organizations_url":"https://api.github.com/users/henrydev10/orgs","repos_url":"https://api.github.com/users/henrydev10/repos","events_url":"https://api.github.com/users/henrydev10/events{/privacy}","received_events_url":"https://api.github.com/users/henrydev10/received_events","type":"User","site_admin":false},"created_at":"2016-10-06T20:55:36Z","updated_at":"2016-10-06T20:55:36Z","body":"Wow, thanks for the enhancement, looks great :)"}}
{"actor":{"display_login":"henrydev10"}}