{"action":"created","issue":{"url":"https://api.github.com/repos/nlp-compromise/nlp_compromise/issues/89","repository_url":"https://api.github.com/repos/nlp-compromise/nlp_compromise","labels_url":"https://api.github.com/repos/nlp-compromise/nlp_compromise/issues/89/labels{/name}","comments_url":"https://api.github.com/repos/nlp-compromise/nlp_compromise/issues/89/comments","events_url":"https://api.github.com/repos/nlp-compromise/nlp_compromise/issues/89/events","html_url":"https://github.com/nlp-compromise/nlp_compromise/issues/89","id":131808013,"number":89,"title":"Consistent interface","user":{"login":"creatorrr","id":931887,"avatar_url":"https://avatars.githubusercontent.com/u/931887?v=3","gravatar_id":"","url":"https://api.github.com/users/creatorrr","html_url":"https://github.com/creatorrr","followers_url":"https://api.github.com/users/creatorrr/followers","following_url":"https://api.github.com/users/creatorrr/following{/other_user}","gists_url":"https://api.github.com/users/creatorrr/gists{/gist_id}","starred_url":"https://api.github.com/users/creatorrr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/creatorrr/subscriptions","organizations_url":"https://api.github.com/users/creatorrr/orgs","repos_url":"https://api.github.com/users/creatorrr/repos","events_url":"https://api.github.com/users/creatorrr/events{/privacy}","received_events_url":"https://api.github.com/users/creatorrr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2016-02-06T02:03:19Z","updated_at":"2016-02-10T01:10:01Z","closed_at":null,"body":"Just wanted to get your feedback on API consistency. I think we could do a lot more to make the API consistent and intuitive to use. Some examples:\r\n\r\n * `to_past`, `to_present`, `to_past` functions return a mutated wrapped instance but `americanize` and `britishize` return just the changed text. Is that by design?\r\n * `Sentence.to_past` returns the mutated `Sentence` instance but same doesn't apply for `Text.to_past`. (I don't know if there's any way to fix that other than extending Array.prototype)\r\n * Unlike other wrappers, `(new NLP.term(...)).text` is a string property rather than a function.\r\n\r\nAlso, I think it might make things a lot less error-prone if we can make the entire library immutable (that'd break backwards compatibility, maybe offer an immutability helper?)"},"comment":{"url":"https://api.github.com/repos/nlp-compromise/nlp_compromise/issues/comments/182155087","html_url":"https://github.com/nlp-compromise/nlp_compromise/issues/89#issuecomment-182155087","issue_url":"https://api.github.com/repos/nlp-compromise/nlp_compromise/issues/89","id":182155087,"user":{"login":"creatorrr","id":931887,"avatar_url":"https://avatars.githubusercontent.com/u/931887?v=3","gravatar_id":"","url":"https://api.github.com/users/creatorrr","html_url":"https://github.com/creatorrr","followers_url":"https://api.github.com/users/creatorrr/followers","following_url":"https://api.github.com/users/creatorrr/following{/other_user}","gists_url":"https://api.github.com/users/creatorrr/gists{/gist_id}","starred_url":"https://api.github.com/users/creatorrr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/creatorrr/subscriptions","organizations_url":"https://api.github.com/users/creatorrr/orgs","repos_url":"https://api.github.com/users/creatorrr/repos","events_url":"https://api.github.com/users/creatorrr/events{/privacy}","received_events_url":"https://api.github.com/users/creatorrr/received_events","type":"User","site_admin":false},"created_at":"2016-02-10T01:10:01Z","updated_at":"2016-02-10T01:10:01Z","body":"I'll start hammering out PRs then!"}}