-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Naming standards #61
Comments
Underscore at the start of the method marks an internal method which also has effect in the stack trace shim that is reliant on function names on firefox. |
Thanks. Also, what's with |
Those are different functions with |
The name on this function doesn't make sense though that should be changed: https://github.com/petkaantonov/bluebird/blob/master/src/promise.js#L1132-L1163 |
Thanks. Feel free to close this with the commit :) |
Hey, I just wanted to ask why the code does things like:
Promise$_all
Promise$_All
andPromise$all
?Is the naming (lower case/upper case) here an important convention?
Can we rename these to something more meaningful?
The text was updated successfully, but these errors were encountered: