{"action":"created","comment":{"url":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/comments/82981731","pull_request_review_id":3850393,"id":82981731,"diff_hunk":"@@ -39,20 +36,17 @@ public CellViewModel(Game _game, int cell)\n \n         protected void NotifyPropertyChanged(String info)\n         {\n-            if (PropertyChanged != null)\n-            {\n-                PropertyChanged(this, new PropertyChangedEventArgs(info));\n-            }\n+            PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(info));\n         }\n \n         public bool CanExecute(object parameter)\n         {\n-            return (_game.isValidPosition(cell));\n+            return (_game.isValidPosition(Cell));\n         }\n \n         public void Execute(object parameter)\n         {\n-            _game.makeMove(cell);\n+            _game.makeMove(Cell);\n             CellMark = 'x';","path":"CellViewModel.cs","position":null,"original_position":51,"commit_id":"e4035add099c224b931e5234d17a51adf6c5b5fa","original_commit_id":"e4035add099c224b931e5234d17a51adf6c5b5fa","user":{"login":"dirv","id":307047,"avatar_url":"https://avatars.githubusercontent.com/u/307047?v=3","gravatar_id":"","url":"https://api.github.com/users/dirv","html_url":"https://github.com/dirv","followers_url":"https://api.github.com/users/dirv/followers","following_url":"https://api.github.com/users/dirv/following{/other_user}","gists_url":"https://api.github.com/users/dirv/gists{/gist_id}","starred_url":"https://api.github.com/users/dirv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dirv/subscriptions","organizations_url":"https://api.github.com/users/dirv/orgs","repos_url":"https://api.github.com/users/dirv/repos","events_url":"https://api.github.com/users/dirv/events{/privacy}","received_events_url":"https://api.github.com/users/dirv/received_events","type":"User","site_admin":false},"body":"This line looks wrong to me (CellMark = 'x'), but at the same time you don't actually need a setter for CellMark at all - just ask the _game for the mark at that position.","created_at":"2016-10-12T10:56:54Z","updated_at":"2016-10-12T10:56:54Z","html_url":"https://github.com/willcurry/WPF-TTT/pull/1#discussion_r82981731","pull_request_url":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/1","_links":{"self":{"href":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/comments/82981731"},"html":{"href":"https://github.com/willcurry/WPF-TTT/pull/1#discussion_r82981731"},"pull_request":{"href":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/1"}}},"pull_request":{"url":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/1","id":88991551,"html_url":"https://github.com/willcurry/WPF-TTT/pull/1","diff_url":"https://github.com/willcurry/WPF-TTT/pull/1.diff","patch_url":"https://github.com/willcurry/WPF-TTT/pull/1.patch","issue_url":"https://api.github.com/repos/willcurry/WPF-TTT/issues/1","number":1,"state":"open","locked":false,"title":"Implement INotfiyPropertyChanged","user":{"login":"willcurry","id":11460313,"avatar_url":"https://avatars.githubusercontent.com/u/11460313?v=3","gravatar_id":"","url":"https://api.github.com/users/willcurry","html_url":"https://github.com/willcurry","followers_url":"https://api.github.com/users/willcurry/followers","following_url":"https://api.github.com/users/willcurry/following{/other_user}","gists_url":"https://api.github.com/users/willcurry/gists{/gist_id}","starred_url":"https://api.github.com/users/willcurry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/willcurry/subscriptions","organizations_url":"https://api.github.com/users/willcurry/orgs","repos_url":"https://api.github.com/users/willcurry/repos","events_url":"https://api.github.com/users/willcurry/events{/privacy}","received_events_url":"https://api.github.com/users/willcurry/received_events","type":"User","site_admin":false},"body":"","created_at":"2016-10-12T10:33:27Z","updated_at":"2016-10-12T10:56:54Z","closed_at":null,"merged_at":null,"merge_commit_sha":"fd477d927aa2f9d3ba44a143629a9a126e2a8dcc","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/1/commits","review_comments_url":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/1/comments","review_comment_url":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/comments{/number}","comments_url":"https://api.github.com/repos/willcurry/WPF-TTT/issues/1/comments","statuses_url":"https://api.github.com/repos/willcurry/WPF-TTT/statuses/e4035add099c224b931e5234d17a51adf6c5b5fa","head":{"label":"willcurry:MVVM","ref":"MVVM","sha":"e4035add099c224b931e5234d17a51adf6c5b5fa","user":{"login":"willcurry","id":11460313,"avatar_url":"https://avatars.githubusercontent.com/u/11460313?v=3","gravatar_id":"","url":"https://api.github.com/users/willcurry","html_url":"https://github.com/willcurry","followers_url":"https://api.github.com/users/willcurry/followers","following_url":"https://api.github.com/users/willcurry/following{/other_user}","gists_url":"https://api.github.com/users/willcurry/gists{/gist_id}","starred_url":"https://api.github.com/users/willcurry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/willcurry/subscriptions","organizations_url":"https://api.github.com/users/willcurry/orgs","repos_url":"https://api.github.com/users/willcurry/repos","events_url":"https://api.github.com/users/willcurry/events{/privacy}","received_events_url":"https://api.github.com/users/willcurry/received_events","type":"User","site_admin":false},"repo":{"id":70680718,"name":"WPF-TTT","full_name":"willcurry/WPF-TTT","owner":{"login":"willcurry","id":11460313,"avatar_url":"https://avatars.githubusercontent.com/u/11460313?v=3","gravatar_id":"","url":"https://api.github.com/users/willcurry","html_url":"https://github.com/willcurry","followers_url":"https://api.github.com/users/willcurry/followers","following_url":"https://api.github.com/users/willcurry/following{/other_user}","gists_url":"https://api.github.com/users/willcurry/gists{/gist_id}","starred_url":"https://api.github.com/users/willcurry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/willcurry/subscriptions","organizations_url":"https://api.github.com/users/willcurry/orgs","repos_url":"https://api.github.com/users/willcurry/repos","events_url":"https://api.github.com/users/willcurry/events{/privacy}","received_events_url":"https://api.github.com/users/willcurry/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/willcurry/WPF-TTT","description":"[C#/XAML] Tic Tac Toe","fork":false,"url":"https://api.github.com/repos/willcurry/WPF-TTT","forks_url":"https://api.github.com/repos/willcurry/WPF-TTT/forks","keys_url":"https://api.github.com/repos/willcurry/WPF-TTT/keys{/key_id}","collaborators_url":"https://api.github.com/repos/willcurry/WPF-TTT/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/willcurry/WPF-TTT/teams","hooks_url":"https://api.github.com/repos/willcurry/WPF-TTT/hooks","issue_events_url":"https://api.github.com/repos/willcurry/WPF-TTT/issues/events{/number}","events_url":"https://api.github.com/repos/willcurry/WPF-TTT/events","assignees_url":"https://api.github.com/repos/willcurry/WPF-TTT/assignees{/user}","branches_url":"https://api.github.com/repos/willcurry/WPF-TTT/branches{/branch}","tags_url":"https://api.github.com/repos/willcurry/WPF-TTT/tags","blobs_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/refs{/sha}","trees_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/trees{/sha}","statuses_url":"https://api.github.com/repos/willcurry/WPF-TTT/statuses/{sha}","languages_url":"https://api.github.com/repos/willcurry/WPF-TTT/languages","stargazers_url":"https://api.github.com/repos/willcurry/WPF-TTT/stargazers","contributors_url":"https://api.github.com/repos/willcurry/WPF-TTT/contributors","subscribers_url":"https://api.github.com/repos/willcurry/WPF-TTT/subscribers","subscription_url":"https://api.github.com/repos/willcurry/WPF-TTT/subscription","commits_url":"https://api.github.com/repos/willcurry/WPF-TTT/commits{/sha}","git_commits_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/commits{/sha}","comments_url":"https://api.github.com/repos/willcurry/WPF-TTT/comments{/number}","issue_comment_url":"https://api.github.com/repos/willcurry/WPF-TTT/issues/comments{/number}","contents_url":"https://api.github.com/repos/willcurry/WPF-TTT/contents/{+path}","compare_url":"https://api.github.com/repos/willcurry/WPF-TTT/compare/{base}...{head}","merges_url":"https://api.github.com/repos/willcurry/WPF-TTT/merges","archive_url":"https://api.github.com/repos/willcurry/WPF-TTT/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/willcurry/WPF-TTT/downloads","issues_url":"https://api.github.com/repos/willcurry/WPF-TTT/issues{/number}","pulls_url":"https://api.github.com/repos/willcurry/WPF-TTT/pulls{/number}","milestones_url":"https://api.github.com/repos/willcurry/WPF-TTT/milestones{/number}","notifications_url":"https://api.github.com/repos/willcurry/WPF-TTT/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/willcurry/WPF-TTT/labels{/name}","releases_url":"https://api.github.com/repos/willcurry/WPF-TTT/releases{/id}","deployments_url":"https://api.github.com/repos/willcurry/WPF-TTT/deployments","created_at":"2016-10-12T08:44:53Z","updated_at":"2016-10-12T08:53:18Z","pushed_at":"2016-10-12T10:42:24Z","git_url":"git://github.com/willcurry/WPF-TTT.git","ssh_url":"git@github.com:willcurry/WPF-TTT.git","clone_url":"https://github.com/willcurry/WPF-TTT.git","svn_url":"https://github.com/willcurry/WPF-TTT","homepage":null,"size":13,"stargazers_count":0,"watchers_count":0,"language":"C#","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"base":{"label":"willcurry:master","ref":"master","sha":"f280fc61b2010b8f88a3dd1aa7c4e0b87e8ae1bc","user":{"login":"willcurry","id":11460313,"avatar_url":"https://avatars.githubusercontent.com/u/11460313?v=3","gravatar_id":"","url":"https://api.github.com/users/willcurry","html_url":"https://github.com/willcurry","followers_url":"https://api.github.com/users/willcurry/followers","following_url":"https://api.github.com/users/willcurry/following{/other_user}","gists_url":"https://api.github.com/users/willcurry/gists{/gist_id}","starred_url":"https://api.github.com/users/willcurry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/willcurry/subscriptions","organizations_url":"https://api.github.com/users/willcurry/orgs","repos_url":"https://api.github.com/users/willcurry/repos","events_url":"https://api.github.com/users/willcurry/events{/privacy}","received_events_url":"https://api.github.com/users/willcurry/received_events","type":"User","site_admin":false},"repo":{"id":70680718,"name":"WPF-TTT","full_name":"willcurry/WPF-TTT","owner":{"login":"willcurry","id":11460313,"avatar_url":"https://avatars.githubusercontent.com/u/11460313?v=3","gravatar_id":"","url":"https://api.github.com/users/willcurry","html_url":"https://github.com/willcurry","followers_url":"https://api.github.com/users/willcurry/followers","following_url":"https://api.github.com/users/willcurry/following{/other_user}","gists_url":"https://api.github.com/users/willcurry/gists{/gist_id}","starred_url":"https://api.github.com/users/willcurry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/willcurry/subscriptions","organizations_url":"https://api.github.com/users/willcurry/orgs","repos_url":"https://api.github.com/users/willcurry/repos","events_url":"https://api.github.com/users/willcurry/events{/privacy}","received_events_url":"https://api.github.com/users/willcurry/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/willcurry/WPF-TTT","description":"[C#/XAML] Tic Tac Toe","fork":false,"url":"https://api.github.com/repos/willcurry/WPF-TTT","forks_url":"https://api.github.com/repos/willcurry/WPF-TTT/forks","keys_url":"https://api.github.com/repos/willcurry/WPF-TTT/keys{/key_id}","collaborators_url":"https://api.github.com/repos/willcurry/WPF-TTT/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/willcurry/WPF-TTT/teams","hooks_url":"https://api.github.com/repos/willcurry/WPF-TTT/hooks","issue_events_url":"https://api.github.com/repos/willcurry/WPF-TTT/issues/events{/number}","events_url":"https://api.github.com/repos/willcurry/WPF-TTT/events","assignees_url":"https://api.github.com/repos/willcurry/WPF-TTT/assignees{/user}","branches_url":"https://api.github.com/repos/willcurry/WPF-TTT/branches{/branch}","tags_url":"https://api.github.com/repos/willcurry/WPF-TTT/tags","blobs_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/refs{/sha}","trees_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/trees{/sha}","statuses_url":"https://api.github.com/repos/willcurry/WPF-TTT/statuses/{sha}","languages_url":"https://api.github.com/repos/willcurry/WPF-TTT/languages","stargazers_url":"https://api.github.com/repos/willcurry/WPF-TTT/stargazers","contributors_url":"https://api.github.com/repos/willcurry/WPF-TTT/contributors","subscribers_url":"https://api.github.com/repos/willcurry/WPF-TTT/subscribers","subscription_url":"https://api.github.com/repos/willcurry/WPF-TTT/subscription","commits_url":"https://api.github.com/repos/willcurry/WPF-TTT/commits{/sha}","git_commits_url":"https://api.github.com/repos/willcurry/WPF-TTT/git/commits{/sha}","comments_url":"https://api.github.com/repos/willcurry/WPF-TTT/comments{/number}","issue_comment_url":"https://api.github.com/repos/willcurry/WPF-TTT/issues/comments{/number}","contents_url":"https://api.github.com/repos/willcurry/WPF-TTT/contents/{+path}","compare_url":"https://api.github.com/repos/willcurry/WPF-TTT/compare/{base}...{head}","merges_url":"https://api.github.com/repos/willcurry/WPF-TTT/merges","archive_url":"https://api.github.com/repos/willcurry/WPF-TTT/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/willcurry/WPF-TTT/downloads","issues_url":"https://api.github.com/repos/willcurry/WPF-TTT/issues{/number}","pulls_url":"https://api.github.com/repos/willcurry/WPF-TTT/pulls{/number}","milestones_url":"https://api.github.com/repos/willcurry/WPF-TTT/milestones{/number}","notifications_url":"https://api.github.com/repos/willcurry/WPF-TTT/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/willcurry/WPF-TTT/labels{/name}","releases_url":"https://api.github.com/repos/willcurry/WPF-TTT/releases{/id}","deployments_url":"https://api.github.com/repos/willcurry/WPF-TTT/deployments","created_at":"2016-10-12T08:44:53Z","updated_at":"2016-10-12T08:53:18Z","pushed_at":"2016-10-12T10:42:24Z","git_url":"git://github.com/willcurry/WPF-TTT.git","ssh_url":"git@github.com:willcurry/WPF-TTT.git","clone_url":"https://github.com/willcurry/WPF-TTT.git","svn_url":"https://github.com/willcurry/WPF-TTT","homepage":null,"size":13,"stargazers_count":0,"watchers_count":0,"language":"C#","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/1"},"html":{"href":"https://github.com/willcurry/WPF-TTT/pull/1"},"issue":{"href":"https://api.github.com/repos/willcurry/WPF-TTT/issues/1"},"comments":{"href":"https://api.github.com/repos/willcurry/WPF-TTT/issues/1/comments"},"review_comments":{"href":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/1/comments"},"review_comment":{"href":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/willcurry/WPF-TTT/pulls/1/commits"},"statuses":{"href":"https://api.github.com/repos/willcurry/WPF-TTT/statuses/e4035add099c224b931e5234d17a51adf6c5b5fa"}}}}
{"actor":{"display_login":"dirv"}}