-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v2: Can of Promises #3729
Open
au-phiware
wants to merge
22
commits into
jashkenas:master
Choose a base branch
from
au-phiware:master
base: master
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
v2: Can of Promises #3729
Commits on Jul 21, 2015
-
Configuration menu - View commit details
-
Copy full SHA for bfa268c - Browse repository at this point
Copy the full SHA bfa268cView commit details -
Configuration menu - View commit details
-
Copy full SHA for e9f019e - Browse repository at this point
Copy the full SHA e9f019eView commit details
Commits on Jul 22, 2015
-
Configuration menu - View commit details
-
Copy full SHA for ecb606f - Browse repository at this point
Copy the full SHA ecb606fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 91227ee - Browse repository at this point
Copy the full SHA 91227eeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9adf677 - Browse repository at this point
Copy the full SHA 9adf677View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5b39cf9 - Browse repository at this point
Copy the full SHA 5b39cf9View commit details
Commits on Jul 24, 2015
-
Configuration menu - View commit details
-
Copy full SHA for bdaa11b - Browse repository at this point
Copy the full SHA bdaa11bView commit details
Commits on Jul 26, 2015
-
Avoid conflict with potential reserved word.
I know ECMA wouldn't break the internet, but one day `catch` might be a reserved word. On the suggestion of @megawac
Configuration menu - View commit details
-
Copy full SHA for ca27d88 - Browse repository at this point
Copy the full SHA ca27d88View commit details -
Don't assume
Promise
is a global.I see no need to refer to `Backbone.Promise` everywhere since the reference from the arguments list isn't going to disappear.
Configuration menu - View commit details
-
Copy full SHA for 53e9555 - Browse repository at this point
Copy the full SHA 53e9555View commit details -
Relax the definition of a
Promise
.The Promise contract with Backbone is only two function, `then` and `catch`, that both return another `Promise`. On the suggestion of @megawac, see jashkenas#3729 (comment)
Configuration menu - View commit details
-
Copy full SHA for 0d4fe5b - Browse repository at this point
Copy the full SHA 0d4fe5bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0bee4c0 - Browse repository at this point
Copy the full SHA 0bee4c0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 403bc44 - Browse repository at this point
Copy the full SHA 403bc44View commit details -
Configuration menu - View commit details
-
Copy full SHA for fb45026 - Browse repository at this point
Copy the full SHA fb45026View commit details -
Configuration menu - View commit details
-
Copy full SHA for 820dd25 - Browse repository at this point
Copy the full SHA 820dd25View commit details
Commits on Jul 27, 2015
-
Configuration menu - View commit details
-
Copy full SHA for 509adab - Browse repository at this point
Copy the full SHA 509adabView commit details -
Configuration menu - View commit details
-
Copy full SHA for 747ffaa - Browse repository at this point
Copy the full SHA 747ffaaView commit details -
Revert "Any constructor or may provide a Promise."
This reverts commit ecb606f at the suggestion of @akre54 see jashkenas#3729 (comment)
Configuration menu - View commit details
-
Copy full SHA for 7eddccd - Browse repository at this point
Copy the full SHA 7eddccdView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2e8c79c - Browse repository at this point
Copy the full SHA 2e8c79cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0833ee7 - Browse repository at this point
Copy the full SHA 0833ee7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 44bec7b - Browse repository at this point
Copy the full SHA 44bec7bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 667f81d - Browse repository at this point
Copy the full SHA 667f81dView commit details
Commits on Aug 8, 2015
-
Configuration menu - View commit details
-
Copy full SHA for 21ab2ee - Browse repository at this point
Copy the full SHA 21ab2eeView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.