{"action":"created","issue":{"url":"https://api.github.com/repos/paldepind/sync-promise/issues/3","repository_url":"https://api.github.com/repos/paldepind/sync-promise","labels_url":"https://api.github.com/repos/paldepind/sync-promise/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/paldepind/sync-promise/issues/3/comments","events_url":"https://api.github.com/repos/paldepind/sync-promise/issues/3/events","html_url":"https://github.com/paldepind/sync-promise/issues/3","id":145053293,"number":3,"title":"Implementing `Promise.resolve`/`Promise.reject` and `then(good, err)`","user":{"login":"brettz9","id":20234,"avatar_url":"https://avatars.githubusercontent.com/u/20234?v=3","gravatar_id":"","url":"https://api.github.com/users/brettz9","html_url":"https://github.com/brettz9","followers_url":"https://api.github.com/users/brettz9/followers","following_url":"https://api.github.com/users/brettz9/following{/other_user}","gists_url":"https://api.github.com/users/brettz9/gists{/gist_id}","starred_url":"https://api.github.com/users/brettz9/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brettz9/subscriptions","organizations_url":"https://api.github.com/users/brettz9/orgs","repos_url":"https://api.github.com/users/brettz9/repos","events_url":"https://api.github.com/users/brettz9/events{/privacy}","received_events_url":"https://api.github.com/users/brettz9/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2016-04-01T00:27:08Z","updated_at":"2016-07-15T22:00:07Z","closed_at":null,"body":"Hi,\r\n\r\nIn your library, it is nice to see there is already a solution for the problem with promises for IndexedDB!\r\n\r\nI would just suggest though that there are a few good reasons to support a couple of the features you rejected (no pun intended).\r\n\r\n`then(good, err)` may be an anti-pattern, but I believe part of the appeal of your library is that it pretty much can work out of the box with existing promises however they are being used, and with a familiar syntax. The fewer the differences, the better, imo.\r\n\r\nImplementing `Promise.resolve` and `Promise.reject` which simply adds a 0 second timeout (or animation frame perhaps) would seem to me to be in the spirit of these \"immediately\" returning. Besides allowing for existing Promises code to avoid the need for changes, it is a convenient practical wrapper of use in and of itself.\r\n\r\nThanks!"},"comment":{"url":"https://api.github.com/repos/paldepind/sync-promise/issues/comments/233080043","html_url":"https://github.com/paldepind/sync-promise/issues/3#issuecomment-233080043","issue_url":"https://api.github.com/repos/paldepind/sync-promise/issues/3","id":233080043,"user":{"login":"zbraniecki","id":449986,"avatar_url":"https://avatars.githubusercontent.com/u/449986?v=3","gravatar_id":"","url":"https://api.github.com/users/zbraniecki","html_url":"https://github.com/zbraniecki","followers_url":"https://api.github.com/users/zbraniecki/followers","following_url":"https://api.github.com/users/zbraniecki/following{/other_user}","gists_url":"https://api.github.com/users/zbraniecki/gists{/gist_id}","starred_url":"https://api.github.com/users/zbraniecki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zbraniecki/subscriptions","organizations_url":"https://api.github.com/users/zbraniecki/orgs","repos_url":"https://api.github.com/users/zbraniecki/repos","events_url":"https://api.github.com/users/zbraniecki/events{/privacy}","received_events_url":"https://api.github.com/users/zbraniecki/received_events","type":"User","site_admin":false},"created_at":"2016-07-15T22:00:07Z","updated_at":"2016-07-15T22:00:07Z","body":"Cool! Did you also remove the limitation on not allowing to chain synchronous thens?"}}
{"actor":{"display_login":"zbraniecki"}}