{"action":"created","issue":{"url":"https://api.github.com/repos/aspnet/Configuration/issues/273","labels_url":"https://api.github.com/repos/aspnet/Configuration/issues/273/labels{/name}","comments_url":"https://api.github.com/repos/aspnet/Configuration/issues/273/comments","events_url":"https://api.github.com/repos/aspnet/Configuration/issues/273/events","html_url":"https://github.com/aspnet/Configuration/pull/273","id":100912989,"number":273,"title":"API review feedback for ConfigurationBinder","user":{"login":"HaoK","id":6537861,"avatar_url":"https://avatars.githubusercontent.com/u/6537861?v=3","gravatar_id":"","url":"https://api.github.com/users/HaoK","html_url":"https://github.com/HaoK","followers_url":"https://api.github.com/users/HaoK/followers","following_url":"https://api.github.com/users/HaoK/following{/other_user}","gists_url":"https://api.github.com/users/HaoK/gists{/gist_id}","starred_url":"https://api.github.com/users/HaoK/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HaoK/subscriptions","organizations_url":"https://api.github.com/users/HaoK/orgs","repos_url":"https://api.github.com/users/HaoK/repos","events_url":"https://api.github.com/users/HaoK/events{/privacy}","received_events_url":"https://api.github.com/users/HaoK/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/aspnet/Configuration/labels/cla-already-signed","name":"cla-already-signed","color":"009800"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-08-14T02:11:47Z","updated_at":"2015-10-05T18:20:38Z","closed_at":"2015-08-15T18:54:18Z","pull_request":{"url":"https://api.github.com/repos/aspnet/Configuration/pulls/273","html_url":"https://github.com/aspnet/Configuration/pull/273","diff_url":"https://github.com/aspnet/Configuration/pull/273.diff","patch_url":"https://github.com/aspnet/Configuration/pull/273.patch"},"body":"Fixes https://github.com/aspnet/Configuration/issues/236\r\n\r\nI took the liberty to clean some stuff up and we don't actually need generics on bind since we don't care what the type is I don't think do we?  We are using reflection.  We only needed the Type when we were new'ing them up in the overload we killed.  Callee now is responsible\r\n\r\ncc @divega @kirthik "},"comment":{"url":"https://api.github.com/repos/aspnet/Configuration/issues/comments/145621668","html_url":"https://github.com/aspnet/Configuration/pull/273#issuecomment-145621668","issue_url":"https://api.github.com/repos/aspnet/Configuration/issues/273","id":145621668,"user":{"login":"aggieben","id":1321,"avatar_url":"https://avatars.githubusercontent.com/u/1321?v=3","gravatar_id":"","url":"https://api.github.com/users/aggieben","html_url":"https://github.com/aggieben","followers_url":"https://api.github.com/users/aggieben/followers","following_url":"https://api.github.com/users/aggieben/following{/other_user}","gists_url":"https://api.github.com/users/aggieben/gists{/gist_id}","starred_url":"https://api.github.com/users/aggieben/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aggieben/subscriptions","organizations_url":"https://api.github.com/users/aggieben/orgs","repos_url":"https://api.github.com/users/aggieben/repos","events_url":"https://api.github.com/users/aggieben/events{/privacy}","received_events_url":"https://api.github.com/users/aggieben/received_events","type":"User","site_admin":false},"created_at":"2015-10-05T18:20:38Z","updated_at":"2015-10-05T18:20:38Z","body":"@divega @HaoK FWIW, I like the sugar that lets me bind and allocate all at once, ala `var settings = _config.Bind(new Settings())`.  The generic version was also just fine."}}