{"action":"created","issue":{"url":"https://api.github.com/repos/thesheps/plang/issues/14","labels_url":"https://api.github.com/repos/thesheps/plang/issues/14/labels{/name}","comments_url":"https://api.github.com/repos/thesheps/plang/issues/14/comments","events_url":"https://api.github.com/repos/thesheps/plang/issues/14/events","html_url":"https://github.com/thesheps/plang/pull/14","id":116995119,"number":14,"title":"Factory pattern spike","user":{"login":"thesheps","id":2959799,"avatar_url":"https://avatars.githubusercontent.com/u/2959799?v=3","gravatar_id":"","url":"https://api.github.com/users/thesheps","html_url":"https://github.com/thesheps","followers_url":"https://api.github.com/users/thesheps/followers","following_url":"https://api.github.com/users/thesheps/following{/other_user}","gists_url":"https://api.github.com/users/thesheps/gists{/gist_id}","starred_url":"https://api.github.com/users/thesheps/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thesheps/subscriptions","organizations_url":"https://api.github.com/users/thesheps/orgs","repos_url":"https://api.github.com/users/thesheps/repos","events_url":"https://api.github.com/users/thesheps/events{/privacy}","received_events_url":"https://api.github.com/users/thesheps/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-11-15T14:31:56Z","updated_at":"2015-11-16T10:58:15Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/thesheps/plang/pulls/14","html_url":"https://github.com/thesheps/plang/pull/14","diff_url":"https://github.com/thesheps/plang/pull/14.diff","patch_url":"https://github.com/thesheps/plang/pull/14.patch"},"body":"@john-chapman - Introducing the factory pattern so as to minimise coupling.  A little refactoring here and there.  Again open for debate!"},"comment":{"url":"https://api.github.com/repos/thesheps/plang/issues/comments/156992424","html_url":"https://github.com/thesheps/plang/pull/14#issuecomment-156992424","issue_url":"https://api.github.com/repos/thesheps/plang/issues/14","id":156992424,"user":{"login":"thesheps","id":2959799,"avatar_url":"https://avatars.githubusercontent.com/u/2959799?v=3","gravatar_id":"","url":"https://api.github.com/users/thesheps","html_url":"https://github.com/thesheps","followers_url":"https://api.github.com/users/thesheps/followers","following_url":"https://api.github.com/users/thesheps/following{/other_user}","gists_url":"https://api.github.com/users/thesheps/gists{/gist_id}","starred_url":"https://api.github.com/users/thesheps/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thesheps/subscriptions","organizations_url":"https://api.github.com/users/thesheps/orgs","repos_url":"https://api.github.com/users/thesheps/repos","events_url":"https://api.github.com/users/thesheps/events{/privacy}","received_events_url":"https://api.github.com/users/thesheps/received_events","type":"User","site_admin":false},"created_at":"2015-11-16T10:58:15Z","updated_at":"2015-11-16T10:58:15Z","body":"That's fine - Happy to close this pull request with your challenge.  My\nonly concern is that ExpressionNode ends up knowing a lot about both\noperands and operators.  If it starts to exhibit behaviours of multiple use\ncases, I might re-create a new branch to this effect.  Sound ok? :)\n"}}