{"action":"created","comment":{"url":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/comments/209439791","pull_request_review_id":145459526,"id":209439791,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIwOTQzOTc5MQ==","diff_hunk":"@@ -0,0 +1,201 @@\n+package org.minions.devfund.noemiguzman.battleship;\n+\n+/**\n+ * Ocean class.\n+ */\n+public class Ocean {\n+    private static final int SIZE_OCEAN = 20;\n+    private static final int NRO_SHIPS = 13;\n+    private Ship[][] ships = new Ship[SIZE_OCEAN][SIZE_OCEAN];\n+    private int shotsFired;\n+    private int hitCount;\n+    private int shipsSunk;\n+\n+    /**\n+     * Creates an empty ocean (fills the ships array with a bunch of EmptySea instances).\n+     * Also initializes any game variables, such as how many shots have been fired.\n+     */\n+    public Ocean() {\n+        shotsFired = 0;\n+        hitCount = 0;\n+        shipsSunk = 0;\n+        for (int i = 0; i < SIZE_OCEAN; i++) {\n+            for (int j = 0; j < SIZE_OCEAN; j++) {\n+                EmptySea emptySea = new EmptySea();\n+                emptySea.placeShipAt(i, j, true, this);\n+            }\n+        }\n+    }\n+\n+    /**\n+     * Place all randomly on the (initially empty) ocean.\n+     * Place larger ships before smaller ones, or you may end up with no legal place to put a large ship.\n+     */\n+    public void placeAllShipsRandomly() {\n+        Ship[] ships =  newShips();","path":"src/main/java/org/minions/devfund/noemiguzman/battleship/Ocean.java","position":35,"original_position":35,"commit_id":"11bd23499cd59acc9a5b628cb86fc87dd9095b63","original_commit_id":"11bd23499cd59acc9a5b628cb86fc87dd9095b63","user":{"login":"carledriss","id":3298869,"node_id":"MDQ6VXNlcjMyOTg4Njk=","avatar_url":"https://avatars2.githubusercontent.com/u/3298869?v=4","gravatar_id":"","url":"https://api.github.com/users/carledriss","html_url":"https://github.com/carledriss","followers_url":"https://api.github.com/users/carledriss/followers","following_url":"https://api.github.com/users/carledriss/following{/other_user}","gists_url":"https://api.github.com/users/carledriss/gists{/gist_id}","starred_url":"https://api.github.com/users/carledriss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/carledriss/subscriptions","organizations_url":"https://api.github.com/users/carledriss/orgs","repos_url":"https://api.github.com/users/carledriss/repos","events_url":"https://api.github.com/users/carledriss/events{/privacy}","received_events_url":"https://api.github.com/users/carledriss/received_events","type":"User","site_admin":false},"body":"![MAJOR](https://sonarsource.github.io/sonar-github/severity-major.png 'Severity: MAJOR') Rename \"ships\" which hides the field declared at line 9. [![rule](https://sonarsource.github.io/sonar-github/rule.png)](https://sonarcloud.io/coding_rules#rule_key=squid%3AHiddenFieldCheck)\n","created_at":"2018-08-11T23:41:24Z","updated_at":"2018-08-11T23:41:24Z","html_url":"https://github.com/MinionsGroup/devfund-h/pull/61#discussion_r209439791","pull_request_url":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/61","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/comments/209439791"},"html":{"href":"https://github.com/MinionsGroup/devfund-h/pull/61#discussion_r209439791"},"pull_request":{"href":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/61"}}},"pull_request":{"url":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/61","id":207797358,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA3Nzk3MzU4","html_url":"https://github.com/MinionsGroup/devfund-h/pull/61","diff_url":"https://github.com/MinionsGroup/devfund-h/pull/61.diff","patch_url":"https://github.com/MinionsGroup/devfund-h/pull/61.patch","issue_url":"https://api.github.com/repos/MinionsGroup/devfund-h/issues/61","number":61,"state":"open","locked":false,"title":"Noemi/hw4battleship","user":{"login":"noemiguzo","id":11005527,"node_id":"MDQ6VXNlcjExMDA1NTI3","avatar_url":"https://avatars3.githubusercontent.com/u/11005527?v=4","gravatar_id":"","url":"https://api.github.com/users/noemiguzo","html_url":"https://github.com/noemiguzo","followers_url":"https://api.github.com/users/noemiguzo/followers","following_url":"https://api.github.com/users/noemiguzo/following{/other_user}","gists_url":"https://api.github.com/users/noemiguzo/gists{/gist_id}","starred_url":"https://api.github.com/users/noemiguzo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/noemiguzo/subscriptions","organizations_url":"https://api.github.com/users/noemiguzo/orgs","repos_url":"https://api.github.com/users/noemiguzo/repos","events_url":"https://api.github.com/users/noemiguzo/events{/privacy}","received_events_url":"https://api.github.com/users/noemiguzo/received_events","type":"User","site_admin":false},"body":"","created_at":"2018-08-11T23:39:16Z","updated_at":"2018-08-11T23:41:24Z","closed_at":null,"merged_at":null,"merge_commit_sha":"b0568f9c5b383e38873e1bdda6e0161e788e594a","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/61/commits","review_comments_url":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/61/comments","review_comment_url":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/comments{/number}","comments_url":"https://api.github.com/repos/MinionsGroup/devfund-h/issues/61/comments","statuses_url":"https://api.github.com/repos/MinionsGroup/devfund-h/statuses/11bd23499cd59acc9a5b628cb86fc87dd9095b63","head":{"label":"MinionsGroup:noemi/HW4battleship","ref":"noemi/HW4battleship","sha":"11bd23499cd59acc9a5b628cb86fc87dd9095b63","user":{"login":"MinionsGroup","id":38253777,"node_id":"MDEyOk9yZ2FuaXphdGlvbjM4MjUzNzc3","avatar_url":"https://avatars2.githubusercontent.com/u/38253777?v=4","gravatar_id":"","url":"https://api.github.com/users/MinionsGroup","html_url":"https://github.com/MinionsGroup","followers_url":"https://api.github.com/users/MinionsGroup/followers","following_url":"https://api.github.com/users/MinionsGroup/following{/other_user}","gists_url":"https://api.github.com/users/MinionsGroup/gists{/gist_id}","starred_url":"https://api.github.com/users/MinionsGroup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MinionsGroup/subscriptions","organizations_url":"https://api.github.com/users/MinionsGroup/orgs","repos_url":"https://api.github.com/users/MinionsGroup/repos","events_url":"https://api.github.com/users/MinionsGroup/events{/privacy}","received_events_url":"https://api.github.com/users/MinionsGroup/received_events","type":"Organization","site_admin":false},"repo":{"id":139185720,"node_id":"MDEwOlJlcG9zaXRvcnkxMzkxODU3MjA=","name":"devfund-h","full_name":"MinionsGroup/devfund-h","owner":{"login":"MinionsGroup","id":38253777,"node_id":"MDEyOk9yZ2FuaXphdGlvbjM4MjUzNzc3","avatar_url":"https://avatars2.githubusercontent.com/u/38253777?v=4","gravatar_id":"","url":"https://api.github.com/users/MinionsGroup","html_url":"https://github.com/MinionsGroup","followers_url":"https://api.github.com/users/MinionsGroup/followers","following_url":"https://api.github.com/users/MinionsGroup/following{/other_user}","gists_url":"https://api.github.com/users/MinionsGroup/gists{/gist_id}","starred_url":"https://api.github.com/users/MinionsGroup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MinionsGroup/subscriptions","organizations_url":"https://api.github.com/users/MinionsGroup/orgs","repos_url":"https://api.github.com/users/MinionsGroup/repos","events_url":"https://api.github.com/users/MinionsGroup/events{/privacy}","received_events_url":"https://api.github.com/users/MinionsGroup/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/MinionsGroup/devfund-h","description":null,"fork":false,"url":"https://api.github.com/repos/MinionsGroup/devfund-h","forks_url":"https://api.github.com/repos/MinionsGroup/devfund-h/forks","keys_url":"https://api.github.com/repos/MinionsGroup/devfund-h/keys{/key_id}","collaborators_url":"https://api.github.com/repos/MinionsGroup/devfund-h/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/MinionsGroup/devfund-h/teams","hooks_url":"https://api.github.com/repos/MinionsGroup/devfund-h/hooks","issue_events_url":"https://api.github.com/repos/MinionsGroup/devfund-h/issues/events{/number}","events_url":"https://api.github.com/repos/MinionsGroup/devfund-h/events","assignees_url":"https://api.github.com/repos/MinionsGroup/devfund-h/assignees{/user}","branches_url":"https://api.github.com/repos/MinionsGroup/devfund-h/branches{/branch}","tags_url":"https://api.github.com/repos/MinionsGroup/devfund-h/tags","blobs_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/refs{/sha}","trees_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/trees{/sha}","statuses_url":"https://api.github.com/repos/MinionsGroup/devfund-h/statuses/{sha}","languages_url":"https://api.github.com/repos/MinionsGroup/devfund-h/languages","stargazers_url":"https://api.github.com/repos/MinionsGroup/devfund-h/stargazers","contributors_url":"https://api.github.com/repos/MinionsGroup/devfund-h/contributors","subscribers_url":"https://api.github.com/repos/MinionsGroup/devfund-h/subscribers","subscription_url":"https://api.github.com/repos/MinionsGroup/devfund-h/subscription","commits_url":"https://api.github.com/repos/MinionsGroup/devfund-h/commits{/sha}","git_commits_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/commits{/sha}","comments_url":"https://api.github.com/repos/MinionsGroup/devfund-h/comments{/number}","issue_comment_url":"https://api.github.com/repos/MinionsGroup/devfund-h/issues/comments{/number}","contents_url":"https://api.github.com/repos/MinionsGroup/devfund-h/contents/{+path}","compare_url":"https://api.github.com/repos/MinionsGroup/devfund-h/compare/{base}...{head}","merges_url":"https://api.github.com/repos/MinionsGroup/devfund-h/merges","archive_url":"https://api.github.com/repos/MinionsGroup/devfund-h/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/MinionsGroup/devfund-h/downloads","issues_url":"https://api.github.com/repos/MinionsGroup/devfund-h/issues{/number}","pulls_url":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls{/number}","milestones_url":"https://api.github.com/repos/MinionsGroup/devfund-h/milestones{/number}","notifications_url":"https://api.github.com/repos/MinionsGroup/devfund-h/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/MinionsGroup/devfund-h/labels{/name}","releases_url":"https://api.github.com/repos/MinionsGroup/devfund-h/releases{/id}","deployments_url":"https://api.github.com/repos/MinionsGroup/devfund-h/deployments","created_at":"2018-06-29T19:02:39Z","updated_at":"2018-08-11T18:13:38Z","pushed_at":"2018-08-11T23:39:16Z","git_url":"git://github.com/MinionsGroup/devfund-h.git","ssh_url":"git@github.com:MinionsGroup/devfund-h.git","clone_url":"https://github.com/MinionsGroup/devfund-h.git","svn_url":"https://github.com/MinionsGroup/devfund-h","homepage":null,"size":283,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":17,"license":null,"forks":0,"open_issues":17,"watchers":0,"default_branch":"develop"}},"base":{"label":"MinionsGroup:develop","ref":"develop","sha":"007c334a1b005923ac9a44f55660fcabbb59fc4b","user":{"login":"MinionsGroup","id":38253777,"node_id":"MDEyOk9yZ2FuaXphdGlvbjM4MjUzNzc3","avatar_url":"https://avatars2.githubusercontent.com/u/38253777?v=4","gravatar_id":"","url":"https://api.github.com/users/MinionsGroup","html_url":"https://github.com/MinionsGroup","followers_url":"https://api.github.com/users/MinionsGroup/followers","following_url":"https://api.github.com/users/MinionsGroup/following{/other_user}","gists_url":"https://api.github.com/users/MinionsGroup/gists{/gist_id}","starred_url":"https://api.github.com/users/MinionsGroup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MinionsGroup/subscriptions","organizations_url":"https://api.github.com/users/MinionsGroup/orgs","repos_url":"https://api.github.com/users/MinionsGroup/repos","events_url":"https://api.github.com/users/MinionsGroup/events{/privacy}","received_events_url":"https://api.github.com/users/MinionsGroup/received_events","type":"Organization","site_admin":false},"repo":{"id":139185720,"node_id":"MDEwOlJlcG9zaXRvcnkxMzkxODU3MjA=","name":"devfund-h","full_name":"MinionsGroup/devfund-h","owner":{"login":"MinionsGroup","id":38253777,"node_id":"MDEyOk9yZ2FuaXphdGlvbjM4MjUzNzc3","avatar_url":"https://avatars2.githubusercontent.com/u/38253777?v=4","gravatar_id":"","url":"https://api.github.com/users/MinionsGroup","html_url":"https://github.com/MinionsGroup","followers_url":"https://api.github.com/users/MinionsGroup/followers","following_url":"https://api.github.com/users/MinionsGroup/following{/other_user}","gists_url":"https://api.github.com/users/MinionsGroup/gists{/gist_id}","starred_url":"https://api.github.com/users/MinionsGroup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MinionsGroup/subscriptions","organizations_url":"https://api.github.com/users/MinionsGroup/orgs","repos_url":"https://api.github.com/users/MinionsGroup/repos","events_url":"https://api.github.com/users/MinionsGroup/events{/privacy}","received_events_url":"https://api.github.com/users/MinionsGroup/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/MinionsGroup/devfund-h","description":null,"fork":false,"url":"https://api.github.com/repos/MinionsGroup/devfund-h","forks_url":"https://api.github.com/repos/MinionsGroup/devfund-h/forks","keys_url":"https://api.github.com/repos/MinionsGroup/devfund-h/keys{/key_id}","collaborators_url":"https://api.github.com/repos/MinionsGroup/devfund-h/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/MinionsGroup/devfund-h/teams","hooks_url":"https://api.github.com/repos/MinionsGroup/devfund-h/hooks","issue_events_url":"https://api.github.com/repos/MinionsGroup/devfund-h/issues/events{/number}","events_url":"https://api.github.com/repos/MinionsGroup/devfund-h/events","assignees_url":"https://api.github.com/repos/MinionsGroup/devfund-h/assignees{/user}","branches_url":"https://api.github.com/repos/MinionsGroup/devfund-h/branches{/branch}","tags_url":"https://api.github.com/repos/MinionsGroup/devfund-h/tags","blobs_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/refs{/sha}","trees_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/trees{/sha}","statuses_url":"https://api.github.com/repos/MinionsGroup/devfund-h/statuses/{sha}","languages_url":"https://api.github.com/repos/MinionsGroup/devfund-h/languages","stargazers_url":"https://api.github.com/repos/MinionsGroup/devfund-h/stargazers","contributors_url":"https://api.github.com/repos/MinionsGroup/devfund-h/contributors","subscribers_url":"https://api.github.com/repos/MinionsGroup/devfund-h/subscribers","subscription_url":"https://api.github.com/repos/MinionsGroup/devfund-h/subscription","commits_url":"https://api.github.com/repos/MinionsGroup/devfund-h/commits{/sha}","git_commits_url":"https://api.github.com/repos/MinionsGroup/devfund-h/git/commits{/sha}","comments_url":"https://api.github.com/repos/MinionsGroup/devfund-h/comments{/number}","issue_comment_url":"https://api.github.com/repos/MinionsGroup/devfund-h/issues/comments{/number}","contents_url":"https://api.github.com/repos/MinionsGroup/devfund-h/contents/{+path}","compare_url":"https://api.github.com/repos/MinionsGroup/devfund-h/compare/{base}...{head}","merges_url":"https://api.github.com/repos/MinionsGroup/devfund-h/merges","archive_url":"https://api.github.com/repos/MinionsGroup/devfund-h/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/MinionsGroup/devfund-h/downloads","issues_url":"https://api.github.com/repos/MinionsGroup/devfund-h/issues{/number}","pulls_url":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls{/number}","milestones_url":"https://api.github.com/repos/MinionsGroup/devfund-h/milestones{/number}","notifications_url":"https://api.github.com/repos/MinionsGroup/devfund-h/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/MinionsGroup/devfund-h/labels{/name}","releases_url":"https://api.github.com/repos/MinionsGroup/devfund-h/releases{/id}","deployments_url":"https://api.github.com/repos/MinionsGroup/devfund-h/deployments","created_at":"2018-06-29T19:02:39Z","updated_at":"2018-08-11T18:13:38Z","pushed_at":"2018-08-11T23:39:16Z","git_url":"git://github.com/MinionsGroup/devfund-h.git","ssh_url":"git@github.com:MinionsGroup/devfund-h.git","clone_url":"https://github.com/MinionsGroup/devfund-h.git","svn_url":"https://github.com/MinionsGroup/devfund-h","homepage":null,"size":283,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":17,"license":null,"forks":0,"open_issues":17,"watchers":0,"default_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/61"},"html":{"href":"https://github.com/MinionsGroup/devfund-h/pull/61"},"issue":{"href":"https://api.github.com/repos/MinionsGroup/devfund-h/issues/61"},"comments":{"href":"https://api.github.com/repos/MinionsGroup/devfund-h/issues/61/comments"},"review_comments":{"href":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/61/comments"},"review_comment":{"href":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/MinionsGroup/devfund-h/pulls/61/commits"},"statuses":{"href":"https://api.github.com/repos/MinionsGroup/devfund-h/statuses/11bd23499cd59acc9a5b628cb86fc87dd9095b63"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"carledriss"}}