{"action":"created","comment":{"url":"https://api.github.com/repos/healthify/guides/pulls/comments/125636197","pull_request_review_id":48043749,"id":125636197,"diff_hunk":"@@ -215,24 +215,34 @@ HTML\n * Don't use a reset button for forms.\n * Prefer cancel links to cancel buttons.\n * Use `<button>` tags over `<a>` tags for actions.\n+* Use semantic tags where applicable. \n \n CSS\n ---\n \n * Use Sass.\n-* Use [Autoprefixer][autoprefixer] to generate vendor prefixes based on the\n-  project-specific browser support that is needed.\n-\n-[autoprefixer]: https://github.com/postcss/autoprefixer\n \n Sass\n ----\n \n-* Prefer `overflow: auto` to `overflow: scroll`, because `scroll` will always\n-  display scrollbars outside of OS X, even when content fits in the container.\n * Use `image-url` and `font-url`, not `url`, so the asset pipeline will re-write\n   the correct paths to assets.\n-* Prefer mixins to `@extend`.\n+* Prefer mixins over `@extend`.\n+* Prefer reusable classes over mixins.\n+* Avoid using ID selectors.\n+* Avoid element selectors.\n+* Avoid decendant selectors, or any highly specific selectors when possible.\n+* Avoid nesting","path":"best-practices/README.md","position":37,"original_position":37,"commit_id":"91fff2e6b45d90e9ff760191f4805ecc996b4200","original_commit_id":"91fff2e6b45d90e9ff760191f4805ecc996b4200","user":{"login":"georgebrock","id":35861,"avatar_url":"https://avatars2.githubusercontent.com/u/35861?v=3","gravatar_id":"","url":"https://api.github.com/users/georgebrock","html_url":"https://github.com/georgebrock","followers_url":"https://api.github.com/users/georgebrock/followers","following_url":"https://api.github.com/users/georgebrock/following{/other_user}","gists_url":"https://api.github.com/users/georgebrock/gists{/gist_id}","starred_url":"https://api.github.com/users/georgebrock/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/georgebrock/subscriptions","organizations_url":"https://api.github.com/users/georgebrock/orgs","repos_url":"https://api.github.com/users/georgebrock/repos","events_url":"https://api.github.com/users/georgebrock/events{/privacy}","received_events_url":"https://api.github.com/users/georgebrock/received_events","type":"User","site_admin":false},"body":"Tiny nitpick: most lines end with a `.`, but this one doesn't (also 242, and 243)","created_at":"2017-07-05T13:02:48Z","updated_at":"2017-07-05T13:03:40Z","html_url":"https://github.com/healthify/guides/pull/14#discussion_r125636197","pull_request_url":"https://api.github.com/repos/healthify/guides/pulls/14","_links":{"self":{"href":"https://api.github.com/repos/healthify/guides/pulls/comments/125636197"},"html":{"href":"https://github.com/healthify/guides/pull/14#discussion_r125636197"},"pull_request":{"href":"https://api.github.com/repos/healthify/guides/pulls/14"}}},"pull_request":{"url":"https://api.github.com/repos/healthify/guides/pulls/14","id":128528020,"html_url":"https://github.com/healthify/guides/pull/14","diff_url":"https://github.com/healthify/guides/pull/14.diff","patch_url":"https://github.com/healthify/guides/pull/14.patch","issue_url":"https://api.github.com/repos/healthify/guides/issues/14","number":14,"state":"open","locked":false,"title":"Update best practices with CSS guidelines","user":{"login":"aaronagray","id":1110019,"avatar_url":"https://avatars1.githubusercontent.com/u/1110019?v=3","gravatar_id":"","url":"https://api.github.com/users/aaronagray","html_url":"https://github.com/aaronagray","followers_url":"https://api.github.com/users/aaronagray/followers","following_url":"https://api.github.com/users/aaronagray/following{/other_user}","gists_url":"https://api.github.com/users/aaronagray/gists{/gist_id}","starred_url":"https://api.github.com/users/aaronagray/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aaronagray/subscriptions","organizations_url":"https://api.github.com/users/aaronagray/orgs","repos_url":"https://api.github.com/users/aaronagray/repos","events_url":"https://api.github.com/users/aaronagray/events{/privacy}","received_events_url":"https://api.github.com/users/aaronagray/received_events","type":"User","site_admin":false},"body":"Including this because they are exactly the things we look out for during Code Review. Further, we point to this repo in the hiring code challenge and want to set applicants up for success by making these things explicit, as we have ruled people out for getting to many of these things wrong in the past.","created_at":"2017-07-02T17:34:57Z","updated_at":"2017-07-05T13:03:40Z","closed_at":null,"merged_at":null,"merge_commit_sha":"7cb62691c72f40b1c4393f57e8760767f700b4b2","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/healthify/guides/pulls/14/commits","review_comments_url":"https://api.github.com/repos/healthify/guides/pulls/14/comments","review_comment_url":"https://api.github.com/repos/healthify/guides/pulls/comments{/number}","comments_url":"https://api.github.com/repos/healthify/guides/issues/14/comments","statuses_url":"https://api.github.com/repos/healthify/guides/statuses/91fff2e6b45d90e9ff760191f4805ecc996b4200","head":{"label":"healthify:patch-best-practices","ref":"patch-best-practices","sha":"91fff2e6b45d90e9ff760191f4805ecc996b4200","user":{"login":"healthify","id":4150341,"avatar_url":"https://avatars0.githubusercontent.com/u/4150341?v=3","gravatar_id":"","url":"https://api.github.com/users/healthify","html_url":"https://github.com/healthify","followers_url":"https://api.github.com/users/healthify/followers","following_url":"https://api.github.com/users/healthify/following{/other_user}","gists_url":"https://api.github.com/users/healthify/gists{/gist_id}","starred_url":"https://api.github.com/users/healthify/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/healthify/subscriptions","organizations_url":"https://api.github.com/users/healthify/orgs","repos_url":"https://api.github.com/users/healthify/repos","events_url":"https://api.github.com/users/healthify/events{/privacy}","received_events_url":"https://api.github.com/users/healthify/received_events","type":"Organization","site_admin":false},"repo":{"id":67049920,"name":"guides","full_name":"healthify/guides","owner":{"login":"healthify","id":4150341,"avatar_url":"https://avatars0.githubusercontent.com/u/4150341?v=3","gravatar_id":"","url":"https://api.github.com/users/healthify","html_url":"https://github.com/healthify","followers_url":"https://api.github.com/users/healthify/followers","following_url":"https://api.github.com/users/healthify/following{/other_user}","gists_url":"https://api.github.com/users/healthify/gists{/gist_id}","starred_url":"https://api.github.com/users/healthify/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/healthify/subscriptions","organizations_url":"https://api.github.com/users/healthify/orgs","repos_url":"https://api.github.com/users/healthify/repos","events_url":"https://api.github.com/users/healthify/events{/privacy}","received_events_url":"https://api.github.com/users/healthify/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/healthify/guides","description":"Guides for Coding Like a Healthifyte.","fork":false,"url":"https://api.github.com/repos/healthify/guides","forks_url":"https://api.github.com/repos/healthify/guides/forks","keys_url":"https://api.github.com/repos/healthify/guides/keys{/key_id}","collaborators_url":"https://api.github.com/repos/healthify/guides/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/healthify/guides/teams","hooks_url":"https://api.github.com/repos/healthify/guides/hooks","issue_events_url":"https://api.github.com/repos/healthify/guides/issues/events{/number}","events_url":"https://api.github.com/repos/healthify/guides/events","assignees_url":"https://api.github.com/repos/healthify/guides/assignees{/user}","branches_url":"https://api.github.com/repos/healthify/guides/branches{/branch}","tags_url":"https://api.github.com/repos/healthify/guides/tags","blobs_url":"https://api.github.com/repos/healthify/guides/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/healthify/guides/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/healthify/guides/git/refs{/sha}","trees_url":"https://api.github.com/repos/healthify/guides/git/trees{/sha}","statuses_url":"https://api.github.com/repos/healthify/guides/statuses/{sha}","languages_url":"https://api.github.com/repos/healthify/guides/languages","stargazers_url":"https://api.github.com/repos/healthify/guides/stargazers","contributors_url":"https://api.github.com/repos/healthify/guides/contributors","subscribers_url":"https://api.github.com/repos/healthify/guides/subscribers","subscription_url":"https://api.github.com/repos/healthify/guides/subscription","commits_url":"https://api.github.com/repos/healthify/guides/commits{/sha}","git_commits_url":"https://api.github.com/repos/healthify/guides/git/commits{/sha}","comments_url":"https://api.github.com/repos/healthify/guides/comments{/number}","issue_comment_url":"https://api.github.com/repos/healthify/guides/issues/comments{/number}","contents_url":"https://api.github.com/repos/healthify/guides/contents/{+path}","compare_url":"https://api.github.com/repos/healthify/guides/compare/{base}...{head}","merges_url":"https://api.github.com/repos/healthify/guides/merges","archive_url":"https://api.github.com/repos/healthify/guides/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/healthify/guides/downloads","issues_url":"https://api.github.com/repos/healthify/guides/issues{/number}","pulls_url":"https://api.github.com/repos/healthify/guides/pulls{/number}","milestones_url":"https://api.github.com/repos/healthify/guides/milestones{/number}","notifications_url":"https://api.github.com/repos/healthify/guides/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/healthify/guides/labels{/name}","releases_url":"https://api.github.com/repos/healthify/guides/releases{/id}","deployments_url":"https://api.github.com/repos/healthify/guides/deployments","created_at":"2016-08-31T15:26:36Z","updated_at":"2016-11-08T20:20:48Z","pushed_at":"2017-07-02T17:34:58Z","git_url":"git://github.com/healthify/guides.git","ssh_url":"git@github.com:healthify/guides.git","clone_url":"https://github.com/healthify/guides.git","svn_url":"https://github.com/healthify/guides","homepage":"","size":64,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":6,"forks":0,"open_issues":6,"watchers":0,"default_branch":"master"}},"base":{"label":"healthify:master","ref":"master","sha":"5ee64825cce8e3157aa90c8be9452bda6ffc0137","user":{"login":"healthify","id":4150341,"avatar_url":"https://avatars0.githubusercontent.com/u/4150341?v=3","gravatar_id":"","url":"https://api.github.com/users/healthify","html_url":"https://github.com/healthify","followers_url":"https://api.github.com/users/healthify/followers","following_url":"https://api.github.com/users/healthify/following{/other_user}","gists_url":"https://api.github.com/users/healthify/gists{/gist_id}","starred_url":"https://api.github.com/users/healthify/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/healthify/subscriptions","organizations_url":"https://api.github.com/users/healthify/orgs","repos_url":"https://api.github.com/users/healthify/repos","events_url":"https://api.github.com/users/healthify/events{/privacy}","received_events_url":"https://api.github.com/users/healthify/received_events","type":"Organization","site_admin":false},"repo":{"id":67049920,"name":"guides","full_name":"healthify/guides","owner":{"login":"healthify","id":4150341,"avatar_url":"https://avatars0.githubusercontent.com/u/4150341?v=3","gravatar_id":"","url":"https://api.github.com/users/healthify","html_url":"https://github.com/healthify","followers_url":"https://api.github.com/users/healthify/followers","following_url":"https://api.github.com/users/healthify/following{/other_user}","gists_url":"https://api.github.com/users/healthify/gists{/gist_id}","starred_url":"https://api.github.com/users/healthify/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/healthify/subscriptions","organizations_url":"https://api.github.com/users/healthify/orgs","repos_url":"https://api.github.com/users/healthify/repos","events_url":"https://api.github.com/users/healthify/events{/privacy}","received_events_url":"https://api.github.com/users/healthify/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/healthify/guides","description":"Guides for Coding Like a Healthifyte.","fork":false,"url":"https://api.github.com/repos/healthify/guides","forks_url":"https://api.github.com/repos/healthify/guides/forks","keys_url":"https://api.github.com/repos/healthify/guides/keys{/key_id}","collaborators_url":"https://api.github.com/repos/healthify/guides/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/healthify/guides/teams","hooks_url":"https://api.github.com/repos/healthify/guides/hooks","issue_events_url":"https://api.github.com/repos/healthify/guides/issues/events{/number}","events_url":"https://api.github.com/repos/healthify/guides/events","assignees_url":"https://api.github.com/repos/healthify/guides/assignees{/user}","branches_url":"https://api.github.com/repos/healthify/guides/branches{/branch}","tags_url":"https://api.github.com/repos/healthify/guides/tags","blobs_url":"https://api.github.com/repos/healthify/guides/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/healthify/guides/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/healthify/guides/git/refs{/sha}","trees_url":"https://api.github.com/repos/healthify/guides/git/trees{/sha}","statuses_url":"https://api.github.com/repos/healthify/guides/statuses/{sha}","languages_url":"https://api.github.com/repos/healthify/guides/languages","stargazers_url":"https://api.github.com/repos/healthify/guides/stargazers","contributors_url":"https://api.github.com/repos/healthify/guides/contributors","subscribers_url":"https://api.github.com/repos/healthify/guides/subscribers","subscription_url":"https://api.github.com/repos/healthify/guides/subscription","commits_url":"https://api.github.com/repos/healthify/guides/commits{/sha}","git_commits_url":"https://api.github.com/repos/healthify/guides/git/commits{/sha}","comments_url":"https://api.github.com/repos/healthify/guides/comments{/number}","issue_comment_url":"https://api.github.com/repos/healthify/guides/issues/comments{/number}","contents_url":"https://api.github.com/repos/healthify/guides/contents/{+path}","compare_url":"https://api.github.com/repos/healthify/guides/compare/{base}...{head}","merges_url":"https://api.github.com/repos/healthify/guides/merges","archive_url":"https://api.github.com/repos/healthify/guides/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/healthify/guides/downloads","issues_url":"https://api.github.com/repos/healthify/guides/issues{/number}","pulls_url":"https://api.github.com/repos/healthify/guides/pulls{/number}","milestones_url":"https://api.github.com/repos/healthify/guides/milestones{/number}","notifications_url":"https://api.github.com/repos/healthify/guides/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/healthify/guides/labels{/name}","releases_url":"https://api.github.com/repos/healthify/guides/releases{/id}","deployments_url":"https://api.github.com/repos/healthify/guides/deployments","created_at":"2016-08-31T15:26:36Z","updated_at":"2016-11-08T20:20:48Z","pushed_at":"2017-07-02T17:34:58Z","git_url":"git://github.com/healthify/guides.git","ssh_url":"git@github.com:healthify/guides.git","clone_url":"https://github.com/healthify/guides.git","svn_url":"https://github.com/healthify/guides","homepage":"","size":64,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":6,"forks":0,"open_issues":6,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/healthify/guides/pulls/14"},"html":{"href":"https://github.com/healthify/guides/pull/14"},"issue":{"href":"https://api.github.com/repos/healthify/guides/issues/14"},"comments":{"href":"https://api.github.com/repos/healthify/guides/issues/14/comments"},"review_comments":{"href":"https://api.github.com/repos/healthify/guides/pulls/14/comments"},"review_comment":{"href":"https://api.github.com/repos/healthify/guides/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/healthify/guides/pulls/14/commits"},"statuses":{"href":"https://api.github.com/repos/healthify/guides/statuses/91fff2e6b45d90e9ff760191f4805ecc996b4200"}}}}
{"actor":{"display_login":"georgebrock"}}