{"action":"created","comment":{"url":"https://api.github.com/repos/alphatica/genotick/pulls/comments/140346050","pull_request_review_id":64407101,"id":140346050,"diff_hunk":"@@ -0,0 +1,56 @@\n+package com.alphatica.genotick.data;\n+\n+import static java.lang.String.format;\n+\n+import java.io.BufferedWriter;\n+import java.io.FileWriter;\n+import java.io.IOException;\n+\n+import com.alphatica.genotick.ui.UserInputOutputFactory;\n+import com.alphatica.genotick.ui.UserOutput;\n+\n+public class FileSystemDataSaver implements DataSaver {\n+    private final UserOutput output = UserInputOutputFactory.getUserOutput();\n+    \n+    @Override\n+    public boolean saveAll(MainAppData data) {\n+        boolean success = true;\n+        for (DataSet set : data.getDataSets()) {","path":"src/main/java/com/alphatica/genotick/data/FileSystemDataSaver.java","position":18,"original_position":18,"commit_id":"4891da56c8b3b1a8c862474bfb0c3a6bf1bbb835","original_commit_id":"4891da56c8b3b1a8c862474bfb0c3a6bf1bbb835","user":{"login":"lukaszwojtow","id":7825540,"avatar_url":"https://avatars3.githubusercontent.com/u/7825540?v=4","gravatar_id":"","url":"https://api.github.com/users/lukaszwojtow","html_url":"https://github.com/lukaszwojtow","followers_url":"https://api.github.com/users/lukaszwojtow/followers","following_url":"https://api.github.com/users/lukaszwojtow/following{/other_user}","gists_url":"https://api.github.com/users/lukaszwojtow/gists{/gist_id}","starred_url":"https://api.github.com/users/lukaszwojtow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lukaszwojtow/subscriptions","organizations_url":"https://api.github.com/users/lukaszwojtow/orgs","repos_url":"https://api.github.com/users/lukaszwojtow/repos","events_url":"https://api.github.com/users/lukaszwojtow/events{/privacy}","received_events_url":"https://api.github.com/users/lukaszwojtow/received_events","type":"User","site_admin":false},"body":"I don't like this. If there is an error - it should throw exception. Exception should be caught higher and served to user in friendly form.","created_at":"2017-09-21T20:07:37Z","updated_at":"2017-09-21T20:07:37Z","html_url":"https://github.com/alphatica/genotick/pull/34#discussion_r140346050","pull_request_url":"https://api.github.com/repos/alphatica/genotick/pulls/34","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/alphatica/genotick/pulls/comments/140346050"},"html":{"href":"https://github.com/alphatica/genotick/pull/34#discussion_r140346050"},"pull_request":{"href":"https://api.github.com/repos/alphatica/genotick/pulls/34"}}},"pull_request":{"url":"https://api.github.com/repos/alphatica/genotick/pulls/34","id":142298324,"html_url":"https://github.com/alphatica/genotick/pull/34","diff_url":"https://github.com/alphatica/genotick/pull/34.diff","patch_url":"https://github.com/alphatica/genotick/pull/34.patch","issue_url":"https://api.github.com/repos/alphatica/genotick/issues/34","number":34,"state":"open","locked":false,"title":"Fix a bug where reversed data can be reversed again. Automatically create reversed data when necessary. Separate DataSet saving from Reversal class. Refactor Reversal functionality.","user":{"login":"xezon","id":4720891,"avatar_url":"https://avatars1.githubusercontent.com/u/4720891?v=4","gravatar_id":"","url":"https://api.github.com/users/xezon","html_url":"https://github.com/xezon","followers_url":"https://api.github.com/users/xezon/followers","following_url":"https://api.github.com/users/xezon/following{/other_user}","gists_url":"https://api.github.com/users/xezon/gists{/gist_id}","starred_url":"https://api.github.com/users/xezon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xezon/subscriptions","organizations_url":"https://api.github.com/users/xezon/orgs","repos_url":"https://api.github.com/users/xezon/repos","events_url":"https://api.github.com/users/xezon/events{/privacy}","received_events_url":"https://api.github.com/users/xezon/received_events","type":"User","site_admin":false},"body":"","created_at":"2017-09-21T09:39:19Z","updated_at":"2017-09-21T20:07:37Z","closed_at":null,"merged_at":null,"merge_commit_sha":"05423b1bdd6205689f918768ca43ec82670ad36a","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/alphatica/genotick/pulls/34/commits","review_comments_url":"https://api.github.com/repos/alphatica/genotick/pulls/34/comments","review_comment_url":"https://api.github.com/repos/alphatica/genotick/pulls/comments{/number}","comments_url":"https://api.github.com/repos/alphatica/genotick/issues/34/comments","statuses_url":"https://api.github.com/repos/alphatica/genotick/statuses/4891da56c8b3b1a8c862474bfb0c3a6bf1bbb835","head":{"label":"xezon:data_improvements_pr2","ref":"data_improvements_pr2","sha":"4891da56c8b3b1a8c862474bfb0c3a6bf1bbb835","user":{"login":"xezon","id":4720891,"avatar_url":"https://avatars1.githubusercontent.com/u/4720891?v=4","gravatar_id":"","url":"https://api.github.com/users/xezon","html_url":"https://github.com/xezon","followers_url":"https://api.github.com/users/xezon/followers","following_url":"https://api.github.com/users/xezon/following{/other_user}","gists_url":"https://api.github.com/users/xezon/gists{/gist_id}","starred_url":"https://api.github.com/users/xezon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xezon/subscriptions","organizations_url":"https://api.github.com/users/xezon/orgs","repos_url":"https://api.github.com/users/xezon/repos","events_url":"https://api.github.com/users/xezon/events{/privacy}","received_events_url":"https://api.github.com/users/xezon/received_events","type":"User","site_admin":false},"repo":{"id":102758525,"name":"genotick","full_name":"xezon/genotick","owner":{"login":"xezon","id":4720891,"avatar_url":"https://avatars1.githubusercontent.com/u/4720891?v=4","gravatar_id":"","url":"https://api.github.com/users/xezon","html_url":"https://github.com/xezon","followers_url":"https://api.github.com/users/xezon/followers","following_url":"https://api.github.com/users/xezon/following{/other_user}","gists_url":"https://api.github.com/users/xezon/gists{/gist_id}","starred_url":"https://api.github.com/users/xezon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xezon/subscriptions","organizations_url":"https://api.github.com/users/xezon/orgs","repos_url":"https://api.github.com/users/xezon/repos","events_url":"https://api.github.com/users/xezon/events{/privacy}","received_events_url":"https://api.github.com/users/xezon/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/xezon/genotick","description":"Artificial Intelligence Data Analysis","fork":true,"url":"https://api.github.com/repos/xezon/genotick","forks_url":"https://api.github.com/repos/xezon/genotick/forks","keys_url":"https://api.github.com/repos/xezon/genotick/keys{/key_id}","collaborators_url":"https://api.github.com/repos/xezon/genotick/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/xezon/genotick/teams","hooks_url":"https://api.github.com/repos/xezon/genotick/hooks","issue_events_url":"https://api.github.com/repos/xezon/genotick/issues/events{/number}","events_url":"https://api.github.com/repos/xezon/genotick/events","assignees_url":"https://api.github.com/repos/xezon/genotick/assignees{/user}","branches_url":"https://api.github.com/repos/xezon/genotick/branches{/branch}","tags_url":"https://api.github.com/repos/xezon/genotick/tags","blobs_url":"https://api.github.com/repos/xezon/genotick/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/xezon/genotick/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/xezon/genotick/git/refs{/sha}","trees_url":"https://api.github.com/repos/xezon/genotick/git/trees{/sha}","statuses_url":"https://api.github.com/repos/xezon/genotick/statuses/{sha}","languages_url":"https://api.github.com/repos/xezon/genotick/languages","stargazers_url":"https://api.github.com/repos/xezon/genotick/stargazers","contributors_url":"https://api.github.com/repos/xezon/genotick/contributors","subscribers_url":"https://api.github.com/repos/xezon/genotick/subscribers","subscription_url":"https://api.github.com/repos/xezon/genotick/subscription","commits_url":"https://api.github.com/repos/xezon/genotick/commits{/sha}","git_commits_url":"https://api.github.com/repos/xezon/genotick/git/commits{/sha}","comments_url":"https://api.github.com/repos/xezon/genotick/comments{/number}","issue_comment_url":"https://api.github.com/repos/xezon/genotick/issues/comments{/number}","contents_url":"https://api.github.com/repos/xezon/genotick/contents/{+path}","compare_url":"https://api.github.com/repos/xezon/genotick/compare/{base}...{head}","merges_url":"https://api.github.com/repos/xezon/genotick/merges","archive_url":"https://api.github.com/repos/xezon/genotick/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/xezon/genotick/downloads","issues_url":"https://api.github.com/repos/xezon/genotick/issues{/number}","pulls_url":"https://api.github.com/repos/xezon/genotick/pulls{/number}","milestones_url":"https://api.github.com/repos/xezon/genotick/milestones{/number}","notifications_url":"https://api.github.com/repos/xezon/genotick/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/xezon/genotick/labels{/name}","releases_url":"https://api.github.com/repos/xezon/genotick/releases{/id}","deployments_url":"https://api.github.com/repos/xezon/genotick/deployments","created_at":"2017-09-07T16:08:15Z","updated_at":"2017-09-07T16:08:17Z","pushed_at":"2017-09-21T09:36:19Z","git_url":"git://github.com/xezon/genotick.git","ssh_url":"git@github.com:xezon/genotick.git","clone_url":"https://github.com/xezon/genotick.git","svn_url":"https://github.com/xezon/genotick","homepage":null,"size":4632,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"alphatica:master","ref":"master","sha":"a21122d2265c1257105c7ec9c0c2e1e0067314b0","user":{"login":"alphatica","id":14149772,"avatar_url":"https://avatars2.githubusercontent.com/u/14149772?v=4","gravatar_id":"","url":"https://api.github.com/users/alphatica","html_url":"https://github.com/alphatica","followers_url":"https://api.github.com/users/alphatica/followers","following_url":"https://api.github.com/users/alphatica/following{/other_user}","gists_url":"https://api.github.com/users/alphatica/gists{/gist_id}","starred_url":"https://api.github.com/users/alphatica/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alphatica/subscriptions","organizations_url":"https://api.github.com/users/alphatica/orgs","repos_url":"https://api.github.com/users/alphatica/repos","events_url":"https://api.github.com/users/alphatica/events{/privacy}","received_events_url":"https://api.github.com/users/alphatica/received_events","type":"Organization","site_admin":false},"repo":{"id":43782760,"name":"genotick","full_name":"alphatica/genotick","owner":{"login":"alphatica","id":14149772,"avatar_url":"https://avatars2.githubusercontent.com/u/14149772?v=4","gravatar_id":"","url":"https://api.github.com/users/alphatica","html_url":"https://github.com/alphatica","followers_url":"https://api.github.com/users/alphatica/followers","following_url":"https://api.github.com/users/alphatica/following{/other_user}","gists_url":"https://api.github.com/users/alphatica/gists{/gist_id}","starred_url":"https://api.github.com/users/alphatica/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alphatica/subscriptions","organizations_url":"https://api.github.com/users/alphatica/orgs","repos_url":"https://api.github.com/users/alphatica/repos","events_url":"https://api.github.com/users/alphatica/events{/privacy}","received_events_url":"https://api.github.com/users/alphatica/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/alphatica/genotick","description":"Artificial Intelligence Data Analysis","fork":false,"url":"https://api.github.com/repos/alphatica/genotick","forks_url":"https://api.github.com/repos/alphatica/genotick/forks","keys_url":"https://api.github.com/repos/alphatica/genotick/keys{/key_id}","collaborators_url":"https://api.github.com/repos/alphatica/genotick/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/alphatica/genotick/teams","hooks_url":"https://api.github.com/repos/alphatica/genotick/hooks","issue_events_url":"https://api.github.com/repos/alphatica/genotick/issues/events{/number}","events_url":"https://api.github.com/repos/alphatica/genotick/events","assignees_url":"https://api.github.com/repos/alphatica/genotick/assignees{/user}","branches_url":"https://api.github.com/repos/alphatica/genotick/branches{/branch}","tags_url":"https://api.github.com/repos/alphatica/genotick/tags","blobs_url":"https://api.github.com/repos/alphatica/genotick/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/alphatica/genotick/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/alphatica/genotick/git/refs{/sha}","trees_url":"https://api.github.com/repos/alphatica/genotick/git/trees{/sha}","statuses_url":"https://api.github.com/repos/alphatica/genotick/statuses/{sha}","languages_url":"https://api.github.com/repos/alphatica/genotick/languages","stargazers_url":"https://api.github.com/repos/alphatica/genotick/stargazers","contributors_url":"https://api.github.com/repos/alphatica/genotick/contributors","subscribers_url":"https://api.github.com/repos/alphatica/genotick/subscribers","subscription_url":"https://api.github.com/repos/alphatica/genotick/subscription","commits_url":"https://api.github.com/repos/alphatica/genotick/commits{/sha}","git_commits_url":"https://api.github.com/repos/alphatica/genotick/git/commits{/sha}","comments_url":"https://api.github.com/repos/alphatica/genotick/comments{/number}","issue_comment_url":"https://api.github.com/repos/alphatica/genotick/issues/comments{/number}","contents_url":"https://api.github.com/repos/alphatica/genotick/contents/{+path}","compare_url":"https://api.github.com/repos/alphatica/genotick/compare/{base}...{head}","merges_url":"https://api.github.com/repos/alphatica/genotick/merges","archive_url":"https://api.github.com/repos/alphatica/genotick/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/alphatica/genotick/downloads","issues_url":"https://api.github.com/repos/alphatica/genotick/issues{/number}","pulls_url":"https://api.github.com/repos/alphatica/genotick/pulls{/number}","milestones_url":"https://api.github.com/repos/alphatica/genotick/milestones{/number}","notifications_url":"https://api.github.com/repos/alphatica/genotick/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/alphatica/genotick/labels{/name}","releases_url":"https://api.github.com/repos/alphatica/genotick/releases{/id}","deployments_url":"https://api.github.com/repos/alphatica/genotick/deployments","created_at":"2015-10-06T22:36:17Z","updated_at":"2017-09-17T01:38:25Z","pushed_at":"2017-09-21T09:39:20Z","git_url":"git://github.com/alphatica/genotick.git","ssh_url":"git@github.com:alphatica/genotick.git","clone_url":"https://github.com/alphatica/genotick.git","svn_url":"https://github.com/alphatica/genotick","homepage":null,"size":4581,"stargazers_count":114,"watchers_count":114,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":70,"mirror_url":null,"open_issues_count":2,"forks":70,"open_issues":2,"watchers":114,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/alphatica/genotick/pulls/34"},"html":{"href":"https://github.com/alphatica/genotick/pull/34"},"issue":{"href":"https://api.github.com/repos/alphatica/genotick/issues/34"},"comments":{"href":"https://api.github.com/repos/alphatica/genotick/issues/34/comments"},"review_comments":{"href":"https://api.github.com/repos/alphatica/genotick/pulls/34/comments"},"review_comment":{"href":"https://api.github.com/repos/alphatica/genotick/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/alphatica/genotick/pulls/34/commits"},"statuses":{"href":"https://api.github.com/repos/alphatica/genotick/statuses/4891da56c8b3b1a8c862474bfb0c3a6bf1bbb835"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"lukaszwojtow"}}