{"action":"created","issue":{"url":"https://api.github.com/repos/servo/servo/issues/8420","labels_url":"https://api.github.com/repos/servo/servo/issues/8420/labels{/name}","comments_url":"https://api.github.com/repos/servo/servo/issues/8420/comments","events_url":"https://api.github.com/repos/servo/servo/issues/8420/events","html_url":"https://github.com/servo/servo/pull/8420","id":115775414,"number":8420,"title":"Enabled use of FontFamily enum type","user":{"login":"craftytrickster","id":10675565,"avatar_url":"https://avatars.githubusercontent.com/u/10675565?v=3","gravatar_id":"","url":"https://api.github.com/users/craftytrickster","html_url":"https://github.com/craftytrickster","followers_url":"https://api.github.com/users/craftytrickster/followers","following_url":"https://api.github.com/users/craftytrickster/following{/other_user}","gists_url":"https://api.github.com/users/craftytrickster/gists{/gist_id}","starred_url":"https://api.github.com/users/craftytrickster/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/craftytrickster/subscriptions","organizations_url":"https://api.github.com/users/craftytrickster/orgs","repos_url":"https://api.github.com/users/craftytrickster/repos","events_url":"https://api.github.com/users/craftytrickster/events{/privacy}","received_events_url":"https://api.github.com/users/craftytrickster/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/servo/servo/labels/S-needs-rebase","name":"S-needs-rebase","color":"d4c5f9"}],"state":"open","locked":false,"assignee":{"login":"glennw","id":2350882,"avatar_url":"https://avatars.githubusercontent.com/u/2350882?v=3","gravatar_id":"","url":"https://api.github.com/users/glennw","html_url":"https://github.com/glennw","followers_url":"https://api.github.com/users/glennw/followers","following_url":"https://api.github.com/users/glennw/following{/other_user}","gists_url":"https://api.github.com/users/glennw/gists{/gist_id}","starred_url":"https://api.github.com/users/glennw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/glennw/subscriptions","organizations_url":"https://api.github.com/users/glennw/orgs","repos_url":"https://api.github.com/users/glennw/repos","events_url":"https://api.github.com/users/glennw/events{/privacy}","received_events_url":"https://api.github.com/users/glennw/received_events","type":"User","site_admin":false},"milestone":null,"comments":9,"created_at":"2015-11-09T01:18:18Z","updated_at":"2015-11-25T19:18:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/servo/servo/pulls/8420","html_url":"https://github.com/servo/servo/pull/8420","diff_url":"https://github.com/servo/servo/pull/8420.diff","patch_url":"https://github.com/servo/servo/pull/8420.patch"},"body":"https://github.com/servo/servo/issues/8371\r\n\r\nIn addition to replacing loose strings with the FontFamily enum in `font_cache_task.rs`, I also centralized the add_generic_font calls into one single function. If centralizing into one function is not desired or if anything else needs to be changed, please let me know.\n\n<!-- Reviewable:start -->\n[<img src=\"https://reviewable.io/review_button.png\" height=40 alt=\"Review on Reviewable\"/>](https://reviewable.io/reviews/servo/servo/8420)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/servo/servo/issues/comments/159705710","html_url":"https://github.com/servo/servo/pull/8420#issuecomment-159705710","issue_url":"https://api.github.com/repos/servo/servo/issues/8420","id":159705710,"user":{"login":"jdm","id":27658,"avatar_url":"https://avatars.githubusercontent.com/u/27658?v=3","gravatar_id":"","url":"https://api.github.com/users/jdm","html_url":"https://github.com/jdm","followers_url":"https://api.github.com/users/jdm/followers","following_url":"https://api.github.com/users/jdm/following{/other_user}","gists_url":"https://api.github.com/users/jdm/gists{/gist_id}","starred_url":"https://api.github.com/users/jdm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdm/subscriptions","organizations_url":"https://api.github.com/users/jdm/orgs","repos_url":"https://api.github.com/users/jdm/repos","events_url":"https://api.github.com/users/jdm/events{/privacy}","received_events_url":"https://api.github.com/users/jdm/received_events","type":"User","site_admin":false},"created_at":"2015-11-25T19:18:47Z","updated_at":"2015-11-25T19:18:47Z","body":"There shouldn't be any need to close this PR. Any changes to address the previous comments and the merge conflicts can be force pushed to the same branch."}}