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

bacon.js implementation #417

Closed
wants to merge 2 commits into from

Conversation

raimohanska
Copy link

Hi, I wrote a bacon.js implemention of TodoMVC. Might you be interested in including that to the labs section?

Bacon.js is not a framework at all but an FRP lib, but I consider FRP+jQuery a viable alternative to using M* frameworks. The approaches are not even mutually exclusive, as has been shown in the bacon.js-require-backbone example by @pyykkis.

What do yo think?

In any case I want to thank you for the great work with TodoMVC! It's been fun checking out the different approaches and even more fun coding my own solution. Time well spent! Sleep is overrated anyway :)

@passy
Copy link
Member

passy commented Feb 1, 2013

Very cool! I saw bacon.js on Hacker News lately and was wondering how you could apply FRP to something like a todo app.

@passy
Copy link
Member

passy commented Feb 1, 2013

You should make sure to read the Contributing Guidelines regarding the coding style though. :)

@raimohanska
Copy link
Author

Tab indent, semicolons => will fix
jshint warnings => will fix
Variables at top of scope => really doesn't make sense with FRP as there's not a single actual variable (everything's treated as immutable)

would this be acceptable?

@raimohanska
Copy link
Author

Done with the changes. Good now? :)

@raimohanska
Copy link
Author

Demo at http://juhajasatu.com/todomvc

@passy
Copy link
Member

passy commented Feb 20, 2013

@raimohanska Thanks for the submission, but we discussed this internally and decided to pass on this. While interesting, we get a lot of implementations and we can't include them all. We felt that FRP is an interesting approach, but the implementation doesn't fit well with the rest.

However we would really like it if you could add it to our wiki: https://github.com/addyosmani/todomvc/wiki/Other-implementations Thanks for understanding. :)

@passy passy closed this Feb 20, 2013
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.

None yet

2 participants