Skip to content
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

Version 0.1.6 #34

Merged
merged 14 commits into from
Aug 6, 2014
Merged

Version 0.1.6 #34

merged 14 commits into from
Aug 6, 2014

Conversation

spoike
Copy link
Member

@spoike spoike commented Aug 5, 2014

Merging in #27 and #33.
Should also fix #26.

@spoike spoike added this to the 0.1.6 milestone Aug 5, 2014
@spoike
Copy link
Member Author

spoike commented Aug 5, 2014

@bripkens if you'd like, could you please check the implementation I did for #26 on the all.js file? If there is any problem with it, please let me know.

Other than that I think it's ready to release out in the wild.

Any comments @dashed?

function didAllListenablesEmit() {
// reduce cannot be used because it only iterates over *present*
// elements in the array. Initially the Array doesn't contain
// elements. Fore this reason the usage of reduce would always indicate
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo "fore"


for (; i < args.length; ++i) {
listener = args[i];
if (listener === listener) {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

listener === listener

Eh, "derp" on my behalf.

@spoike
Copy link
Member Author

spoike commented Aug 5, 2014

@KyleAMathews, @dashed Done.

I'll be releasing this tomorrow after I got some sleep if nobody objects. :-)

@spoike spoike merged commit 414bfb7 into master Aug 6, 2014
@bripkens
Copy link
Contributor

bripkens commented Aug 6, 2014

Looks great, thanks for the release :)

@spoike spoike deleted the 0.1.6 branch August 6, 2014 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Throw error when stores are chained in a circular loop
4 participants