search

madrobby / semicolon.js

A more secure. stable and reliable version of vapor.js
GNU General Public License v3.0
229 stars 28 forks source link

Add redundancy in favor of losing a few ROFLScale points. #2

Closed alloy closed 7 years ago

alloy commented 12 years ago

Nuff said.

See #1.

madrobby commented 12 years ago

I think this pull request needs some extended discussion before it can go in, as we have plans do release something similar on our commercial Semicolon.js Enterprise Platform™.

alloy commented 12 years ago

Oh I see. Probably better to have such a discussion on HackerNews, though.

drublic commented 12 years ago

This is way to heavy. The complexity of this solution tippled the complexity of the whole library. I really don't like it. Also I've not seen any test for the SEP (Semicolon.js Enterprise Platform™). Could you pleases add them? Thank you!

djcas9 commented 12 years ago

Guys, please stop using semicolons properly. You will hurt the feelings of twitter/bootstrap. https://github.com/twitter/bootstrap/issues/3057 - Also, can you port this to coffeescript.. I can't read javascript.

drublic commented 12 years ago

@mephux Could you please open another issue for the coffee-script port? Thank you!

djcas9 commented 12 years ago

Hey @drublic - I opened a pull request. Thank you! :)

alloy commented 7 years ago

5 years later, it’s become very clear to me that you do not respect all the time I’ve put into this.