{"action":"created","issue":{"url":"https://api.github.com/repos/mdhiggins/sickbeard_mp4_automator/issues/256","repository_url":"https://api.github.com/repos/mdhiggins/sickbeard_mp4_automator","labels_url":"https://api.github.com/repos/mdhiggins/sickbeard_mp4_automator/issues/256/labels{/name}","comments_url":"https://api.github.com/repos/mdhiggins/sickbeard_mp4_automator/issues/256/comments","events_url":"https://api.github.com/repos/mdhiggins/sickbeard_mp4_automator/issues/256/events","html_url":"https://github.com/mdhiggins/sickbeard_mp4_automator/pull/256","id":73102719,"node_id":"MDExOlB1bGxSZXF1ZXN0MzQ2NjUzMTA=","number":256,"title":"manual.py - Adds in a check for output files and skips processing if they exists.","user":{"login":"KurtisVargas","id":11523782,"node_id":"MDQ6VXNlcjExNTIzNzgy","avatar_url":"https://avatars1.githubusercontent.com/u/11523782?v=4","gravatar_id":"","url":"https://api.github.com/users/KurtisVargas","html_url":"https://github.com/KurtisVargas","followers_url":"https://api.github.com/users/KurtisVargas/followers","following_url":"https://api.github.com/users/KurtisVargas/following{/other_user}","gists_url":"https://api.github.com/users/KurtisVargas/gists{/gist_id}","starred_url":"https://api.github.com/users/KurtisVargas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KurtisVargas/subscriptions","organizations_url":"https://api.github.com/users/KurtisVargas/orgs","repos_url":"https://api.github.com/users/KurtisVargas/repos","events_url":"https://api.github.com/users/KurtisVargas/events{/privacy}","received_events_url":"https://api.github.com/users/KurtisVargas/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2015-05-04T19:13:39Z","updated_at":"2018-09-22T07:55:25Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/mdhiggins/sickbeard_mp4_automator/pulls/256","html_url":"https://github.com/mdhiggins/sickbeard_mp4_automator/pull/256","diff_url":"https://github.com/mdhiggins/sickbeard_mp4_automator/pull/256.diff","patch_url":"https://github.com/mdhiggins/sickbeard_mp4_automator/pull/256.patch"},"body":"This is the full debugged version of a patch that I left in a comment a few weeks ago.\n\nThis patch adds a command line option to manual.py that will check if the output file already exists before processing the input file.  Trivial change but this enables two key features:\n1) Manual.py can run multiple times on the same collection of input files to mass update an entire collection.  Great for mass updates or mass conversions.\n2) Networked machines can work simultaneously on the same collection so long as the output-folder is the same.\n"},"comment":{"url":"https://api.github.com/repos/mdhiggins/sickbeard_mp4_automator/issues/comments/423725456","html_url":"https://github.com/mdhiggins/sickbeard_mp4_automator/pull/256#issuecomment-423725456","issue_url":"https://api.github.com/repos/mdhiggins/sickbeard_mp4_automator/issues/256","id":423725456,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMzcyNTQ1Ng==","user":{"login":"lowne","id":6718755,"node_id":"MDQ6VXNlcjY3MTg3NTU=","avatar_url":"https://avatars2.githubusercontent.com/u/6718755?v=4","gravatar_id":"","url":"https://api.github.com/users/lowne","html_url":"https://github.com/lowne","followers_url":"https://api.github.com/users/lowne/followers","following_url":"https://api.github.com/users/lowne/following{/other_user}","gists_url":"https://api.github.com/users/lowne/gists{/gist_id}","starred_url":"https://api.github.com/users/lowne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lowne/subscriptions","organizations_url":"https://api.github.com/users/lowne/orgs","repos_url":"https://api.github.com/users/lowne/repos","events_url":"https://api.github.com/users/lowne/events{/privacy}","received_events_url":"https://api.github.com/users/lowne/received_events","type":"User","site_admin":false},"created_at":"2018-09-22T07:55:25Z","updated_at":"2018-09-22T07:55:25Z","author_association":"NONE","body":"@mdhiggins  *please* merge this. I've stashed-and-applied it for years now, since it's required by my workflow - and before this I *did* experience catastrophic data loss due to something upstream going haywire.\r\n\r\nThe probe approach, while laudable, is an orthogonal feature meant for not-necessarily-overlapping use cases (e.g. it would not have prevented *my* specific situation), and it also has the non-trivial drawback of *not having an implementation yet*. This patch has existed for over 3 years, is straightforward, tested, and - let me repeat - **prevents data loss**."}}
{"actor":{"display_login":"lowne"}}