Sure, let's discuss that at the next meeting.


I personally think the jQuery guide is the most readable of all the major ones, which I decided after researching them as linked here: http://addyosmani.com/blog/javascript-style-guides-and-beautifiers/.  In particular, I think the guidelines on spacing make for super readable code, and overall it is compatible with existing Engine code I have dealt with.  (I am not, for example, suggesting we go with the NPM style guide!)





I should have mentioned earlier that I did discuss code style with Curtis before opting to follow the jQuery style guide.  The result of my quick talk with Curtis is that he was comfortable with me choosing a style for the front end code since I'm writing most of it.  After that conversation I began implementing jQuery, along with their jshintrc.




I hope these notes give you a little more context for that decision!




Ian
—
Sent from Mailbox for iPhone

On Thu, Mar 27, 2014 at 9:57 AM, Faiz Khan <notifications@github.com>
wrote:

>> @@ -0,0 +1,15 @@
>> +{
>> +	"boss": true,
>> +	"curly": true,
>> +	"eqeqeq": true,
>> +	"eqnull": true,
>> +	"expr": true,
>> +	"immed": true,
>> +	"noarg": true,
>> +	"onevar": true,
>> +	"quotmark": "single",
>> +	"smarttabs": true,
>> +	"trailing": true,
>> +	"undef": true,
> I'm mostly saying this from tradition, since there are a number of code format choices made for Engine-Developer as well as Omniwallet that (At least me and Curtis) have stuck with. 
> I mainly would just like to see a reason for enforcing this specific code style over others, instead of being told to do something differently; as an example, we're historically used '==' to weed out corner cases making comparisons, the style guide you've chosen would enforce '===', so it would actually be somewhat maddening to switch back to that and make sure we've used triple equals correctly.
> That sort of thing; but also, if you need a jshint, can't we discuss whats best for the project and merge the existing ones with the one you've chosen?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/LDEngine/Billing/pull/3/files#r11023224