{"action":"created","issue":{"url":"https://api.github.com/repos/thephpleague/container/issues/62","labels_url":"https://api.github.com/repos/thephpleague/container/issues/62/labels{/name}","comments_url":"https://api.github.com/repos/thephpleague/container/issues/62/comments","events_url":"https://api.github.com/repos/thephpleague/container/issues/62/events","html_url":"https://github.com/thephpleague/container/pull/62","id":100864127,"number":62,"title":"Merge v2 Development","user":{"login":"philipobenito","id":2077289,"avatar_url":"https://avatars.githubusercontent.com/u/2077289?v=3","gravatar_id":"","url":"https://api.github.com/users/philipobenito","html_url":"https://github.com/philipobenito","followers_url":"https://api.github.com/users/philipobenito/followers","following_url":"https://api.github.com/users/philipobenito/following{/other_user}","gists_url":"https://api.github.com/users/philipobenito/gists{/gist_id}","starred_url":"https://api.github.com/users/philipobenito/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/philipobenito/subscriptions","organizations_url":"https://api.github.com/users/philipobenito/orgs","repos_url":"https://api.github.com/users/philipobenito/repos","events_url":"https://api.github.com/users/philipobenito/events{/privacy}","received_events_url":"https://api.github.com/users/philipobenito/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-08-13T20:44:25Z","updated_at":"2015-08-14T08:32:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/thephpleague/container/pulls/62","html_url":"https://github.com/thephpleague/container/pull/62","diff_url":"https://github.com/thephpleague/container/pull/62.diff","patch_url":"https://github.com/thephpleague/container/pull/62.patch"},"body":"This PR aims to be a discussion platform for version 2 of the container over the coming 2 weeks before a release is tagged on Friday 28th August 2015.\r\n\r\nIt is incomplete and will be added to over the coming days with a completed test suite and any missing functionality.\r\n\r\nThe main talking points I'd like to discuss are as follows:\r\n\r\n- Should \"auto-wiring\" be removed? I see three options\r\n    1. Remove reflection completely (my preferred option).\r\n    2. Attempt reflection as the last step of object resolution (turned off by default).\r\n    3. Attempt reflection as the last step of object resolution (turned on by default).\r\n\r\n- The question of config. I've had several conversations with people requesting a better way to interact with the container via array type configs. This is something I've decided to include, but, should it be via new methods or by refactoring the current methods so that they can be invoked with different types of arguments then a simple `array_map` can pass the config to them as a callable?\r\n\r\nThere will be more discussion over the coming days, but these two issues are my main concern as they are blockers to finishing the functionality. \r\n\r\nPlease don't provide any edits to the code as of yet without discussion around it but if there is something you'd like to get involved with then please bring it up here so that it can be properly assigned and attributed."},"comment":{"url":"https://api.github.com/repos/thephpleague/container/issues/comments/131024046","html_url":"https://github.com/thephpleague/container/pull/62#issuecomment-131024046","issue_url":"https://api.github.com/repos/thephpleague/container/issues/62","id":131024046,"user":{"login":"localheinz","id":605483,"avatar_url":"https://avatars.githubusercontent.com/u/605483?v=3","gravatar_id":"","url":"https://api.github.com/users/localheinz","html_url":"https://github.com/localheinz","followers_url":"https://api.github.com/users/localheinz/followers","following_url":"https://api.github.com/users/localheinz/following{/other_user}","gists_url":"https://api.github.com/users/localheinz/gists{/gist_id}","starred_url":"https://api.github.com/users/localheinz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/localheinz/subscriptions","organizations_url":"https://api.github.com/users/localheinz/orgs","repos_url":"https://api.github.com/users/localheinz/repos","events_url":"https://api.github.com/users/localheinz/events{/privacy}","received_events_url":"https://api.github.com/users/localheinz/received_events","type":"User","site_admin":false},"created_at":"2015-08-14T08:32:48Z","updated_at":"2015-08-14T08:32:48Z","body":"@philipobenito \r\n\r\n> Should \"auto-wiring\" be removed? I see three options\r\n> \r\n> i. Remove reflection completely (my preferred option).\r\n\r\n:+1:\r\n\r\n> The question of config. I've had several conversations with people requesting a better way to interact with the container via array type configs. This is something I've decided to include, but, should it be via new methods or by refactoring the current methods so that they can be invoked with different types of arguments then a simple array_map can pass the config to them as a callable?\r\n\r\n* Not sure if I'm doing it right, but how about the container itself as a configuration object, similar to Pimple? \r\n* How about being able to access values via dot-notation, for example like this: https://github.com/dflydev/dflydev-dot-access-configuration"}}