The method name of current linq.js is UpperCamelCase.
However, the standard naming convention of JavaScript is lowerCamelCase.
I think I made lowerCamelCase at the time of an initial design is big mistake.

It was announced BUILD that a JavaScript was chosen as one of the languages of WinRT.
Of course the place of the activity of JS including Node.js and MongoDB spreads out more and more.

I decided that I switched to lowerCamelCase as a breaking change.

How does it seem about this?
Tell the yes and no.

Either way is fine with me...if I need to fix my code that doesn't look like a large issue.

Do you expect to release a new version anytime soon?

I find this project very useful. Thank you for your efforts.

Cheers,

Matthew

I'm very much in favor of the lowerCamelCase convention. Sure using UpperCamelCase makes it look more familiar to C# developers, but from the point of view of a pure JavaScript developer it looks very strange. And IDEs likes WebStorm by JetBrains which offer some form of code validation will complain about using constructor functions in a bad way, since UpperCamelCase is usually reserved for constructors. I actually thought about forking linq.js and converting all uppercase functions to lowercase, because it is a bit annoying.

By the way, while you are add it, please add semicolons to the function assignments. 

var Enumerable = function (getEnumerator) {
      this.GetEnumerator = getEnumerator;
};

But don't get me wrong, linq.js represents a stunning effort and it is the best LINQ implementation in JavaScript I have seen  so far. I really have to thank you for your efforts!

Thank you for all of you.
If there is some request, please say.

By the way, when switch to lowerCamelCase,
it is necessary to change a method name.
For example, Do or Catch.
Therefore I want to add adjustment to a method name.

Do -> doAction
Catch -> catchError
Finally -> finallyDo
Return -> make
ToString -> toJoinedString
CascadeDepthFirst -> traverseDepthFirst
CascadeBreadthFirst -> traverseBreadthFirst
BufferWithCount -> buffer
MemoizeAll -> memoize

Is this all right for ToString?
Please say alternative plan.

//

My plannning schedule is
beta release in October, stable in December.

@neuecc, thanks for all your great work!

If you are going to make a significant upgrade to the package, I strongly suggest that you put in a number of changes so that the library can be used with the Closure Compiler.  In my experience, using the Closure Compiler eliminates most of the overheads of using this library and takes away the performance penalty.

I'll find a place to upload my modified version of linq.js (which works with the Closure Compiler) for your reference!

Thanks a GREAT DEAL for such a fabulous piece of work!  It makes JavaScript much less painful to write!

@neuecc, FYI, I have created an issue with an attached file (the modified linq.js) for your reference.  Thanks!

Oh, god. Please no. I beg you.

Don't change to lowerCamel. It's so much more harder to read. 

MS's Reactive Extensions too features the dual methods of Enumerable in UpperCamel and they're got support for WinRT, WinPhone, Silverlight and .NET - they've gone out of the way to make sure it stays all the same. You have no idea how cool it is to combine Rx and Linqjs.

- 

Please leave Enumerable like it is. It's perfect. Ignore the javascript purists. 

A great example would be Three.js (https://github.com/mrdoob/three.js/)

var camera = new THREE.PerspectiveCamera(70, window.innerWidth / window.innerHeight, 1, 1000);

So much easier to read.

asti0 wrote:

A great example would be Three.js (https://github.com/mrdoob/three.js/)

var camera = new THREE.PerspectiveCamera(70, window.innerWidth / window.innerHeight, 1, 1000);

So much easier to read.

The above code is a constructor and such should be named with UpperCamelCasing according standard JavaScript naming conventions. Object members should be named with lowerCamelCasing which Three.js also follows if you have a look at the other source code:-)

This said, I am also in favor of following JavaScript naming conventions and use lowerCamelCasing which all other JS frameworks adhere to.

+1 for following JavaScript conventions.  Also why do you want to rename toString?  This is not a reserved keywork.

Latest RxJs release has made the change to lowerCamel.  +1 for Linq.js to do the same!

@gatapia:  toString is a method on the Object.prototype and, by convention, takes no arguments and converts the object into some sort of string representation.  IMO it would be a bit confusing to have toString() overloads on enumerables which do customizable string concatenation of the sequence.  I'm definitely in favor of renaming it to something different to avoid that confusion.

+1 on camel case.

Please NO. It just makes it harder to port code from C#.

+1 for camelCase. It's always better to stick to language conventions rather than import different conventions from other languages. Even WinRT does this.

+1 for camelCase.

Also, is there any way to shorten Enumerable.From(array)?  Seems wordy compared to other libraries like underscore.

-1

All +1 to change to javascript standard format are crazy!!!

How mutch code you wrote until now?

I think you wrote few lines...

I have an javascript app widely using linq.js And I hope no rewrite the code because some purist people think is not cool or javascript exact format.

Please do not change the different things, accept it. Be tolerant of it.

In the other hand... why waste your valuable time in this "stetic" question when you have a lot of ideas and work to do...