{"action":"created","issue":{"url":"https://api.github.com/repos/nucleic/enaml/issues/187","labels_url":"https://api.github.com/repos/nucleic/enaml/issues/187/labels{/name}","comments_url":"https://api.github.com/repos/nucleic/enaml/issues/187/comments","events_url":"https://api.github.com/repos/nucleic/enaml/issues/187/events","html_url":"https://github.com/nucleic/enaml/pull/187","id":90772189,"number":187,"title":"Added entry point for extensions to add custom factories","user":{"login":"gabrielcnr","id":209902,"avatar_url":"https://avatars.githubusercontent.com/u/209902?v=3","gravatar_id":"","url":"https://api.github.com/users/gabrielcnr","html_url":"https://github.com/gabrielcnr","followers_url":"https://api.github.com/users/gabrielcnr/followers","following_url":"https://api.github.com/users/gabrielcnr/following{/other_user}","gists_url":"https://api.github.com/users/gabrielcnr/gists{/gist_id}","starred_url":"https://api.github.com/users/gabrielcnr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gabrielcnr/subscriptions","organizations_url":"https://api.github.com/users/gabrielcnr/orgs","repos_url":"https://api.github.com/users/gabrielcnr/repos","events_url":"https://api.github.com/users/gabrielcnr/events{/privacy}","received_events_url":"https://api.github.com/users/gabrielcnr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-24T20:23:25Z","updated_at":"2015-06-24T21:50:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nucleic/enaml/pulls/187","html_url":"https://github.com/nucleic/enaml/pull/187","diff_url":"https://github.com/nucleic/enaml/pull/187.diff","patch_url":"https://github.com/nucleic/enaml/pull/187.patch"},"body":"Hey Chris, how are you?\r\n\r\nDo you still remember me? :-)\r\n\r\nSo, I'm proposing this simple change that will already be enough as an extension/entry point to allow enaml plugins to declare custom factories.\r\n\r\nI also created a simple demo project that makes usage of the feature in:\r\nhttps://github.com/gabrielcnr/enaml_hello\r\n\r\nCan you please take a look to see if this can be merged?\r\n\r\nThanks,\r\nGabe"},"comment":{"url":"https://api.github.com/repos/nucleic/enaml/issues/comments/115021823","html_url":"https://github.com/nucleic/enaml/pull/187#issuecomment-115021823","issue_url":"https://api.github.com/repos/nucleic/enaml/issues/187","id":115021823,"user":{"login":"sccolbert","id":137289,"avatar_url":"https://avatars.githubusercontent.com/u/137289?v=3","gravatar_id":"","url":"https://api.github.com/users/sccolbert","html_url":"https://github.com/sccolbert","followers_url":"https://api.github.com/users/sccolbert/followers","following_url":"https://api.github.com/users/sccolbert/following{/other_user}","gists_url":"https://api.github.com/users/sccolbert/gists{/gist_id}","starred_url":"https://api.github.com/users/sccolbert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sccolbert/subscriptions","organizations_url":"https://api.github.com/users/sccolbert/orgs","repos_url":"https://api.github.com/users/sccolbert/repos","events_url":"https://api.github.com/users/sccolbert/events{/privacy}","received_events_url":"https://api.github.com/users/sccolbert/received_events","type":"User","site_admin":false},"created_at":"2015-06-24T21:50:22Z","updated_at":"2015-06-24T21:50:22Z","body":"This is an interesting idea :)\r\n\r\nOne thing I think of though, is that users of the extension will still need to import the Enaml declaration object. The extension could simpy register its factories during the import. What does this proposal gain us over that approach?"}}